While reviewing my code, it appears I forgot two changes in engine.php:

   - replace
   copy("$boltwire/shared/img/favicon.ico", "favicon.ico");
   by
   if (! file_exists('favicon.ico')) copy("$boltwire/shared/img/favicon.ico"
   , "favicon.ico");
   replace
   - // Elininate slashes if php has magic quotes turned on
   by
   // Eliminate slashes if php has magic quotes turned on
   - remove
   if (isset($BOLTstopWatch)) $out = $BOLTstopWatchMsg . $out;
   since we output $BOLTstopWatchMsg anyway via 
   if (inlist('admin', $BOLTgroups)) $out = str_replace("<body>", "<body><div 
   class='stopwatch'>$BOLTstopWatchMsg</div>", $out);
   
   
That's it.
Cheers,
Tiffany


Le lundi 25 mai 2015 09:54:16 UTC+2, Tiffany Grenier a écrit :
>
> Hi, it's me again !
>
> Always looking for a way to easily have a more flexible BOLTwire, I came 
> to notice the following things:
>
>    - adding a mesage such as 
>    var_content: The content of variable $1 is $2
>    would be very helpful for debuggging purposes
>    - in BOLTCless, 
>    if (is_numeric($test0) && is_numeric($test1)) return $test0 < $test1;
>    could be replaced by
>    if (is_number($test0) && is_number($test1)) return $test0 < $test1;
>    since is_numeric was deprecated
>    - Im BOLTCnew, I don't see the use of line
>    if ($c == '') $type = count($BOLTnewCond);
>    If there is none, it should be removed
>    - I suggest to replace all &#39; by &apos; and all &#34; by &quot; in 
>    order to be consitent with the use of &lt; and &gt;
>    - I propose to add a variable {:parent} that only returns the parent 
>    page of the current page (the before-last part). This feature can be 
>    achieve with 3 lines in library.php:
>    $out = str_replace('{+parent}', substr($page,0,strrpos($page,'.')), 
>    $out);
>    needs to be added in BOLTdisplayReplace, 
>    $BOLTvar['#'][$page]['parent'] = substr($page,0,strrpos($page,'.'));
>    needs to be added in BOLTvarCache,
>    and you should replace in BOLTvars
>    if (inlist($field, 
>    ":p,:page,:title,:p0,:p1,:p2,:p3,:p4,:p5,:p6,:p7,:p8,:p9,:data")) 
>    $field = substr($field, 1);
>    by
>    if (inlist($field, 
>    ":p,:page,:title,:parent,:p0,:p1,:p2,:p3,:p4,:p5,:p6,:p7,:p8,:p9,:data"
>    )) $field = substr($field, 1); 
>    - Don't have more than one BOLTmessage per line, in order to improve 
>    readability, and 
>    foreach($arr as $a) $out .= implode(' ', $a) . ' ';
>    into
>    foreach($arr as $a) $out .= implode("\n", $a) . "\n";
>    One could also think about removing exact duplicates (like "source 
>    retrieved for page"... which my appear one hundred times with the data 
>    action, but I have no suggestion for that at the moment
>    - Add a unique flag in the search function
>    - In BOLTsearchResults, replace
>    if ($args['if'] != '' || $args['count'] != '') $outarray = 
>    BOLTsearchResultsWhenCount($outarray, $args);
>    by
>    if ($args['when'] != '' || $args['count'] != '') $outarray = 
>    BOLTsearchResultsWhenCount($outarray, $args);
>    - In BOLTsecureFilter, replace
>    if ($args['case'] != '') {
>     switch ($args['case']) {
>     case upper: $value = strtoupper($value); break;
>     case lower: $value = strtolower($value); break;
>     case initial: $value = ucfirst(strtolower($value)); break;
>     case words: $value = ucwords(strtolower($value)); break;
>     }
>     }
>    by
>    if ($args['case'] != '') {
>     switch ($args['case']) {
>     case 'upper': $value = strtoupper($value); break;
>     case 'lower': $value = strtolower($value); break;
>     case 'initial': $value = ucfirst(strtolower($value)); break;
>     case 'words': $value = ucwords(strtolower($value)); break;
>     }
>     }
>    - In BOLTexists, you should replace
>    if (strpos(dir, '..') !== false) return false;
>    by
>    if (strpos($dir, '..') !== false) return false;
>    - In markups.php, replace
>    MarkUp('style', 'color', 
>    '/&lt;color\=[\'"]?([a-zA-Z0-9\#]{3,9})[\'"]?>(.*?)&lt;\/color>/sie', 
> "BOLTMspandivs('color', 
>    '$1', '$2')"); // <color=red>
>    by 
>    MarkUp('style', 'color', 
>    '/&lt;color\=[\'"]?([a-zA-Z0-9\#]{3,25})[\'"]?>(.*?)&lt;\/color>/sie', 
> "BOLTMspandivs('color', 
>    '$1', '$2')"); // <color=red>
>    because the longest valid color name has 25 characters (see 
>    http://www.w3schools.com/html/html_colornames.asp)
>
>
> Cheers,
> Tiffany
>

-- 
You received this message because you are subscribed to the Google Groups 
"BoltWire" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/boltwire.
For more options, visit https://groups.google.com/d/optout.

Reply via email to