.DISPLAY is causing issues with Dynamic Action

[maxhorizon]

Hi,

I think appending .DISPLAY is not a good idea.
I have tried to user [P_ITEMNAME].DISPLAY in the dynamic actions trigger but it is not working.

I have changed the code in the plugin section to append _DISPLAY instead.
That fixed the issue.

Regards,
Salim

[Sam-Hall]

Fair enough, what I'll do is use a plugin attribute for this and set the default to "_DISPLAY". The idea was just to use something that wasn't a legal page item id so as to avoid conflicts.

Since it's causing other problems, I'll just make it configurable. I'd better wait until after ODTUG competition winners are declared.

[maxhorizon]

Nice.

I loved your plugin by the way.
Keep up the good work.

See you at Kscope

Regards,
Salim

On Mon, May 14, 2012 at 4:47 PM, CaptEgg <
reply@reply.github.com

wrote:

Fair enough, what I'll do is use a plugin attribute for this and set the
default to "_DISPLAY". The idea was just to use something that wasn't a
legal page item id so as to avoid conflicts.

Since it's causing other problems, I'll just make it configurable. I'd
better wait until after ODTUG competition winners are declared.

---

Reply to this email directly or view it on GitHub:

#1 (comment)