lastedit action breaks validation

[minusf@…]

hi there,

{{lastedit show="3"}} breaks validation: & is not escaped.

 http://obiit.org/wikka_patches/lastedit_php.patch is a fixed and somewhat more readable version.

[dartar]

Some code cleanup

[dartar]

Hi, I've modified your version to use a phpdoc header; added a bunch of configurable defaults and i18n strings; more compact code using switch.

As for coding conventions, we tend not to use whitespace in string concatenation.

Also, be careful with table suffixes that must be appended without space to $this->config['table_prefix']

[dartar]

Patch committed in [32]

[minusf@…]

in the phpdoc header, version should be an svn tag, no?

[dartar]

Yes, it will be something like:

@version SVN: $Id$

or

@version SVN: $Rev$ $Author$

this will have to be applied to all code as soon as we reach an agreement on code documentation guidelines.

Please note that this ticket is fixed because the issue you requested (validation problem) is resolved. This doesn't obviously exclude that the code can undergo further modifications before it is released. Please bear in mind that what is closed in the tracker is a specific request, not the development of an action. So you should only reopen an issue if you think that the original request has not been resolved.

[JavaWoman]

Seems to have been reopened by spam removal; closing again...