MODX :: notempty filter not working with placeholder

315

jrotering Reply #1, 1 year, 8 months ago

Link to this post#1

This must have something to do with the parse order or something.

[[!setPlaceholders]] // $modx->setPlaceholder('siteCssOverride', '/path/to/override.css');
[[+siteCssOverride:default=`<link rel="stylesheet" href="[[+siteCssOverride]]" />`]] // this works, outputs <link> element
[[+siteCssOverride:notempty=`<link rel="stylesheet" href="[[+siteCssOverride]]" />`]]//this doesn't work; no ouput

:notempty seems to work fine for TVs and other elements, but not for placeholders. Is the placeholder value somehow empty at the time the filter is applied? Very confusing.

315

jrotering Reply #2, 1 year, 8 months ago

Link to this post#2

This works as a workaround to do what I want (show nothing if empty, show the value inside a <link> element if not empty):

[[+siteCssOverride:if=`[[+siteCssOverride]]`:eq=``:then=``:else=`<link rel="stylesheet" href="/[[+siteCssOverride]]" />`]]

1,869

Mark Hamstra
Senior Developer, MODX Complete Team
MODX, LLC 
Email: support@modx.com
Twitter: @mark_hamstra
Web: modx.com | Twitter: @modxcms | Facebook: modxcms, MODX.Nederland

Also visit my Github profile, personal website and MODX Blog for more MODX goodness.

Mark Hamstra - MODX Complete Team Reply #3, 1 year, 3 months ago

Link to this post#3

Running into the same problem in 2.0.7... while the workaround is a possibility, I'm also wondering why this wont work on placeholders...

A similar (but slightly cleaner imo) workaround could be

[[!+fi.error.email:isnot=``:then=`<br />[[!+fi.error.email]]`]]

4,064

- MODX Senior Developer

splittingred Reply #4, 1 year, 3 months ago

Link to this post#4

You should make those placeholders uncached.

1,869

Mark Hamstra
Senior Developer, MODX Complete Team
MODX, LLC 
Email: support@modx.com
Twitter: @mark_hamstra
Web: modx.com | Twitter: @modxcms | Facebook: modxcms, MODX.Nederland

Also visit my Github profile, personal website and MODX Blog for more MODX goodness.

Mark Hamstra - MODX Complete Team Reply #5, 1 year, 3 months ago

Link to this post#5

I can't talk for James, but it aint working for me uncached either...

4,064

- MODX Senior Developer

splittingred Reply #6, 1 year, 3 months ago

Link to this post#6

[[!+fi.error.email:notempty=`<br />[[!+fi.error.email]]`]]

Should work fine. I've tested that on multiple environments.

2

mikem.uk Reply #7, 4 months, 1 week ago

Link to this post#7

I was struggling to get this working. Seems to be important to put the snippet call that sets the placeholder in the code before :notempty.