Skins /
CleanSimple-Talk
Summary: Talk page for CleanSimple.
Maintainer: Cameron Walker
This space is for User-contributed commentary and notes. Please include your name and a date along with your comment.
PHP5.5 Compatibility Issue
M. Denning, 20170730: Ruleset is throwing the following compatibility error. Please, can this be addressed ASAP? Base site is battzion.org and WebMaster contact is at the bottom. Thanks!
notitle directives B>>>= ! file: /CleanSimple/skin.php, line: 52, pat: /\(:notitle:\)/ei nosearch directives B>>>= ! file: /CleanSimple/skin.php, line: 49, pat: /\(:nosearch:\)/ei notabs directives B>>>= ! file: /CleanSimple/skin.php, line: 47, pat: /\(:notabs:\)/ei notitlegroup directives B>>>= ! file: /CleanSimple/skin.php, line: 51, pat: /\(:notitlegroup:\)/ei noaction directives B>>>= ! file: /CleanSimple/skin.php, line: 45, pat: /\(:noaction:\)/ei noleft directives B>>>= ! file: /CleanSimple/skin.php, line: 41, pat: /\(:noleft:\)/ei noright directives B>>>= ! file: /CleanSimple/skin.php, line: 43, pat: /\(:noright:\)/ei
Update: | I have updated lines 41, 43, 45, 47, 49, 51 and 52. Lines 53 and 54 do not seem to be using the \e option and therefore were not changed to markup_e. |
---|---|
FROM | 041: Markup('noleft', 'directives', '/\\(:noleft:\\)/ei', "SetTmplDisplay('PageLeftFmt',0)"); |
TO | 041: Markup_e('noleft', 'directives', '/\\(:noleft:\\)/i', "SetTmplDisplay('PageLeftFmt',0)"); |
FROM | 043: Markup('noright', 'directives', '/\\(:noright:\\)/ei', "SetTmplDisplay('PageRightFmt',0)"); |
TO | 043: Markup_e('noright', 'directives', '/\\(:noright:\\)/i', "SetTmplDisplay('PageRightFmt',0)"); |
FROM | 045: Markup('noaction', 'directives', '/\\(:noaction:\\)/ei', "SetTmplDisplay('PageActionFmt',0)"); |
TO | 045: Markup_e('noaction', 'directives', '/\\(:noaction:\\)/i', "SetTmplDisplay('PageActionFmt',0)"); |
FROM | 047: Markup('notabs', 'directives', '/\\(:notabs:\\)/ei', "SetTmplDisplay('PageTabsFmt',0)"); |
TO | 047: Markup_e('notabs', 'directives', '/\\(:notabs:\\)/i', "SetTmplDisplay('PageTabsFmt',0)"); |
FROM | 049: Markue('nosearch', 'directives', '/\\(:nosearch:\\)/ei', "SetTmplDisplay('PageSearchFmt',0)"); |
TO | 049: Markup_e('nosearch', 'directives', '/\\(:nosearch:\\)/i', "SetTmplDisplay('PageSearchFmt',0)"); |
FROM | 051: Markup('notitlegroup', 'directives', '/\\(:notitlegroup:\\)/ei', "SetTmplDisplay('PageTitleGroupFmt',0)"); |
TO | 051: Markup_e('notitlegroup', 'directives', '/\\(:notitlegroup:\\)/i', "SetTmplDisplay('PageTitleGroupFmt',0)"); |
FROM | 052: Markup('notitle', 'directives', '/\\(:notitle:\\)/ei', "SetTmplDisplay('PageTitleFmt',0); |
TO | 052: Markup_e('notitle', 'directives', '/\\(:notitle:\\)/i', "SetTmplDisplay('PageTitleFmt',0); |
TakeAway: | This seems to have taken away the error generated in Rule Table. |
AfterThought: | However, I would still be more comfortable if I obtained positive word from the other developers that this was the only updated necessary. |
Talk page for CleanSimple (users?).