legendbox not compatible with 1.6.5

A place to report and discuss bugs - please mention CMSimple-version, server, platform and browser version
cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: legendbox not compatible with 1.6.5

Post by cmb » Wed Feb 04, 2015 11:08 pm

Holger wrote:I was the one voted "never" on that topic for 1.6.5.
To avoid misunderstandings: the change that was introduced with 1.6.5 is the possibility to omit the parentheses and the trailing semicolon. What svasti suggests is to partly revert the change introduced with 1.6, namely that the leading PLUGIN can be replaced with an arbitrary comment.
Holger wrote:IMO we should not try to make everything as much foolproof for inexperienced users as possible.
That might be worth a separate discussion. :) It seems to me one of the main goals of CMSimple_XH was to be more user friendly (especially for inexperienced ones); for instance, at least the original functionality of meta_tags and page_params was not really new -- all of it could be done with CMSimple scripting before. hi_pdScripting and the plugin call are alternatives to make CMSimple scripting simpler for more general cases. Why not improve on this, if possible.
Holger wrote:I'm fine with one (and only one) clear and correct syntax for a plugin-call.
I have to agree. Probably it would have been better to simply disallow PLUGIN: with XH 1.6, and to disallow the parentheses and the trailing semicolon with XH 1.7. That would have broken resp. would break backwards compatibility, but most likely it could (have) be(en) solved by a small upgrade script (update_content.php).

It is not clear for me what to do now. Simply reverting the changes from 1.6.5 would not really solve all issues. Above I gave an example of a plugin call that already would have failed under 1.6.4.
Holger wrote:And if something goes wrong, a short error message should be enough.
ACK. However, currently that is not necessarily possible, because the plugin call still allows arbitrary PHP expressions for the arguments. For instance, the following would give a fatal error instead of nice error message:

Code: Select all

{{{PLUGIN:trim(wurst());}}}
I would prefer to simplify the plugin call to allow only simple values as arguments (more complex needs can use CMSimple scripting), but there has been some opposition when I wanted to do that for XH 1.6.
Christoph M. Becker – Plugins for CMSimple_XH

svasti
Posts: 1651
Joined: Wed Dec 17, 2008 5:08 pm

Re: legendbox not compatible with 1.6.5

Post by svasti » Mon Feb 09, 2015 11:30 am

cmb wrote:What svasti suggests is to partly revert the change introduced with 1.6, namely that the leading PLUGIN can be replaced with an arbitrary comment.
Exactly. Compatibility is kept, features nobody missed before are not introduced. I've put in on the 1.6.6 road map.

Holger
Site Admin
Posts: 3470
Joined: Mon May 19, 2008 7:10 pm
Location: Hessen, Germany

Re: legendbox not compatible with 1.6.5

Post by Holger » Mon Feb 09, 2015 12:07 pm

svasti wrote:
cmb wrote:What svasti suggests is to partly revert the change introduced with 1.6, namely that the leading PLUGIN can be replaced with an arbitrary comment.
Exactly. Compatibility is kept, features nobody missed before are not introduced. I've put in on the 1.6.6 road map.
That's fine with me :) .

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: legendbox not compatible with 1.6.5

Post by cmb » Thu Apr 02, 2015 12:27 pm

See also the related discussion in http://cmsimpleforum.com/viewtopic.php?f=10&t=8644.
Christoph M. Becker – Plugins for CMSimple_XH

Post Reply