CleanSimple-Talk

Summary: Talk page for CleanSimple.
Maintainer: Cameron Walker
Users: (View? / Edit)

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.
FROM041: Markup('noleft', 'directives', '/\\(:noleft:\\)/ei', "SetTmplDisplay('PageLeftFmt',0)");
TO041: Markup_e('noleft', 'directives', '/\\(:noleft:\\)/i', "SetTmplDisplay('PageLeftFmt',0)");
FROM043: Markup('noright', 'directives', '/\\(:noright:\\)/ei', "SetTmplDisplay('PageRightFmt',0)");
TO043: Markup_e('noright', 'directives', '/\\(:noright:\\)/i', "SetTmplDisplay('PageRightFmt',0)");
FROM045: Markup('noaction', 'directives', '/\\(:noaction:\\)/ei', "SetTmplDisplay('PageActionFmt',0)");
TO045: Markup_e('noaction', 'directives', '/\\(:noaction:\\)/i', "SetTmplDisplay('PageActionFmt',0)");
FROM047: Markup('notabs', 'directives', '/\\(:notabs:\\)/ei', "SetTmplDisplay('PageTabsFmt',0)");
TO047: Markup_e('notabs', 'directives', '/\\(:notabs:\\)/i', "SetTmplDisplay('PageTabsFmt',0)");
FROM049: Markue('nosearch', 'directives', '/\\(:nosearch:\\)/ei', "SetTmplDisplay('PageSearchFmt',0)");
TO049: Markup_e('nosearch', 'directives', '/\\(:nosearch:\\)/i', "SetTmplDisplay('PageSearchFmt',0)");
FROM051: Markup('notitlegroup', 'directives', '/\\(:notitlegroup:\\)/ei', "SetTmplDisplay('PageTitleGroupFmt',0)");
TO051: Markup_e('notitlegroup', 'directives', '/\\(:notitlegroup:\\)/i', "SetTmplDisplay('PageTitleGroupFmt',0)");
FROM052: Markup('notitle', 'directives', '/\\(:notitle:\\)/ei', "SetTmplDisplay('PageTitleFmt',0);
TO052: 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?).