-
- 36 Posts
In Newspublisher 3, has anything changed with the `&hoverhelp`-option? I don't get any TV descriptions, `[[+npx.help]]` is always empty since I updated to version 3.
-
- 24,544 Posts
I'm not aware of any changes to that, but I'll try to find time to take a look.
Do you have this in your NewsPublisher tag?
-
- 24,544 Posts
The placeholder should be +npx.help. It gets replaced with this language tag (in the NewsPublisher class file around line 1156):
[[%resource_' . $field . '_help:notags]]
It could be a problem with the :notags filter itself, or a more general problem with a filter used on a language tag.
Is there anything relevant in the MODX Error log?
-
- 24,544 Posts
This line should be around line 1453 of the newspublisher.class.php file (under core/components/newspublisher/model/newspublisher/).
$replace['[[+npx.help]]'] = $this->modx->getOption('hoverhelp', $description, '', true);
I believe it should be:
$replace['[[+npx.help]]'] = $this->modx->getOption('hoverhelp')? $description : '';
It should use the description of the TV as the hover help.
-
- 24,544 Posts
Good catch. I think this would be slightly better:
$replace['[[+npx.help]]'] = $this->modx->getOption('hoverhelp', $this->props, false, true)? $description : '';
-
- 36 Posts
Great! And thanks for your help!
Will this be included in the next release of NewsPublisher?
-
- 24,544 Posts
Absolutely, though it may be a while.
Thanks for discovering the problem and testing the solution.
