Lay-out iPad php versie 7

[Rolf]

Gregor, je kijkt in de CGBlog template "archiveSample"?

[30-Mar-2017 20:47:37 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 173

Algemene opmerking (voor alle lezers):
Weet dat deze "notices" of wel waarschuwingen geen bugs/storingen zijn, en ook niet een direct gevolg zijn van de PHP verandering! Het zijn eigenlijk slordigheidjes die in de vorige opstelling ook ontstonden alleen niet zichtbaar waren. Meestal zijn deze met een simpele config regel te verstoppen... Maar in dit geval - een trage website - kun je dat beter niet doen. Ook iedere foutmelding kost serverpower! Dus vooral bij een grote website beter even wat tijd in stoppen. En je leert er ook altijd van! De volgende website of template die je maakt zal zeker minder "fouten" in het log bevatten.

Een (module)developer zal code iedere keer moeten herzien als gevolg van PHP, jQuery, etc. wijzigingen.
Dit geldt voor websitebouwers ook. Het is goed om eens in het error log te kijken en te zien welke fouten worden getoond.
Je kunt daarmee aanpassingen doen in templates en UDTs waardoor de site blijft werken bij upgrades van de CMS software, server, etc.

Belangrijk is en blijft dat je de server, templates en css schoon houdt, simpelweg: bomen en het bos...
Ik heb er ooit eens dit korte blog over geschreven:
https://www.cmscanbesimple.org/blog/web ... nance-tips

[Gregor]

Ik kijk in zowel de Sample als in Jaar_Maand -template. Vervang ik in de aanroep summarytemplate voor archivetemplate, dan krijg ik op de sitepagina de volgende melding:

Warning: strpos(): Empty needle in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 55

In de template van de foutmelding staat de volgende regel code:

Code: Select all

{if (isset($entry->fieldsbyname.Tags->value)) AND (($entry->fieldsbyname.Tags->value|strpos:$tag) !== false)}

$tag wordt een paar regels erboven als volgt gedefinieerd:

Code: Select all

{$tag = $smarty.get.tag|default:''}

T.a.v. jouw algemene opmerking, bij de settings van PHP in cpanel is de mogelijkheid om logging uit te zetten. M.i. je ogen sluiten voor de werkelijkheid. Maar dat schrijf je eigenlijk ook.

Bij het zoeken naar het trage serverprobleem viel me al op hoe snel de errorlog groeit. Ben meermalen logs tegen gekomen die de grootte van de 5MB overstijgen.

Een site die, zoals in mijn geval, sinds 2006 (!) bestaat, daar is ook mijn ontwikkeling t.a.v. cmsms in te zien. Gebruik van allerhande functionaliteit verandert en het is best een klus om geplaatste content aan te passen aan de nieuwe werkelijkheid als diezelfde content ook nog met de oude werkelijkheid werkt, en je de logfile negeert.

[Rolf]

Haha inderdaad, ik heb mijn buispost site ook al meerdere keren opnieuw opgebouwd. Veel werk. Iedere keer leer je weer wat nieuws en wil je weer wat anders... Meertalige website maken met 3 CMS-en, voor iedere taal 1 was toch wel *mijn* dieptepunt. Ugh......

Met m'n opmerking wil ik anderen die dit draadje lezen bemoedigen om de PHP wel te verhogen als dat kan! CMSMS en de meeste modules zijn geschikt. PHP 7.0 dan, 7.1 weet ik niet zeker... In ieder geval werken nieuwere PHP versies veel sneller.


Back on topic,
volgens mij gaan we een beetje in een rondje Gregor...
Alles werkt nog steeds naar behoren en je hebt geen foutmeldingen op de frontend?
Niet ergens anders een gerichte melding waar we mee verder kunnen?

[Rolf]

Code: Select all

{foreach from=$archivelist item=one}
{if $lastused!=$one.year}
   <div class="articles">
     <h2>Logboek over {$one.year}</h2>
     {CGBlog year=$one.year archivetemplate="Jaar_Maand"}
   </div>
   {assign var='lastused' value=$one.year}
{/if}
{/foreach}

Probeer eens:

Code: Select all

{foreach from=$archivelist item=one}
{if !empty($one.year)}
   {if $lastused != $one.year}
      <div class="articles">
         <h2>Logboek over {$one.year}</h2>
         {CGBlog year=$one.year archivetemplate="Jaar_Maand"}
      </div>
      {assign var='lastused' value=$one.year}
   {/if}
{/if}
{/foreach}

[Gregor]

Helaas Rolf, ook die blijft een hele lijst undefined index geven. Ik heb twee verschillende tekens geplaats, een boven en een onder de {if......} Deze tekens worden geen van beide weergegeven. Dat zou betekenen dat hij helemaal niet in template "Jaar_Maand" komt

[01-Apr-2017 18:57:13 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 165
[01-Apr-2017 18:57:13 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 165
[01-Apr-2017 18:57:13 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 165
[01-Apr-2017 18:57:13 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 165

Zou jij de archive template eens op jouw omgeving willen proberen en kijken wat er in de errorlog (in de root van je site) komt te staan?

[Gregor]

Een klein zijstapje. De tag-optie in CGBlog werkte niet:

Notice: Undefined variable: value in /home/efacti/public_html/uisge-beatha.eu/www/plugins/function.cgblog_tagvalues.php on line 20

Notice: Undefined variable: ausgabe in /home/efacti/public_html/uisge-beatha.eu/www/plugins/function.cgblog_tagvalues.php on line 31

In blauw de aanpassing in de code en in rood de regels waar de foutmelding op verscheen.

<?php
#Helper Script by TemplateMadeSimple

function smarty_function_cgblog_tagvalues($params, &$template)
{
//global $gCms;
$db = cmsms()->GetDb();
$tagfield = $params['item'];
$abfrage= "SELECT value FROM ".cms_db_prefix()."module_cgblog_fieldvals WHERE fielddef_id =". $tagfield;
$ergebnis= $db->Execute($abfrage);

$tags = array();
$value = ""; // needed as of php 7
$ausgabe = ""; // needed as of php 7
while($ergebnis && $eintrag = $ergebnis->FetchRow() )
{
$tags[] = strtolower($eintrag["value"]);
}

foreach($tags as $tag)
{
$value = $tag.", ".$value;
}
$value = substr($value,0,strlen($value)-2);
$value = str_replace(",,",",",$value);

$value = explode(",", $value);
$value = array_unique($value);
foreach($value as $tag)
{
if(trim($tag) != "")
{
$ausgabe = "'".addslashes(ucfirst(trim($tag)))."', ".$ausgabe;
}
}
$ausgabe = substr($ausgabe,0,strlen($ausgabe)-2);
echo $ausgabe;
}

function smarty_cms_help_function_cgblog_tagvalues() {
echo "No help is needed!";
}

function smarty_cms_about_function_cgblog_tagvalues() {
?>
<p>Author: TMS: http://www.templatemadesimple.de</p>
<p>Version: 1.0</p>
<?php
}
?>

hmmm, hoe laat je een kleur in de tekst veranderen?

[Rolf]

https://www.cmscanbesimple.org/gregor

Code: Select all

{CGBlog action='archive'}

Front end error reporting is enabled.

Code: Select all

{$lastused = ''}

{foreach from=$archivelist item=one}
   {if $lastused != $one.year}
      <div class="articles">
         <h2>Logboek over {$one.year}</h2>
         {CGBlog year=$one.year archivetemplate="Jaar_Maand"}
      </div>
      {assign var='lastused' value=$one.year}
   {/if}
{/foreach}

Ik moest alleen de eerste regel toevoegen

[Gregor]

Met jouw toevoeging krijg ik nu wel output die past bij wat er in de archive template 'Sample' is gecodeerd, echter in de errorlog (en op de website) krijg ik de foutmelding:

02-Apr-2017 18:48:29 UTC] PHP Warning: strpos(): Empty needle in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 55
[02-Apr-2017 18:48:29 UTC] PHP Notice: Undefined index: title in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 137
[02-Apr-2017 18:48:29 UTC] PHP Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 137
[02-Apr-2017 18:48:29 UTC] PHP Warning: strpos(): Empty needle in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 55
[02-Apr-2017 18:48:29 UTC] PHP Notice: Undefined index: title in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 137
[02-Apr-2017 18:48:29 UTC] PHP Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 137

Het code fragment uit Tag_CGBlog waar 'strpos' inzit is:

Code: Select all

<!-- Tag_CGBlog -->
{* Start CGBlog Display origineel Tag_CGBlog Template *}
{if isset($error)}
    {cgerror}
        {$error}
    {/cgerror}
{/if}
{$tag = $smarty.get.tag|default:''}
<h5>Artikelen gerelateerd aan {$tag}</h5>
{foreach from=$items item=entry}
    {* check url for param tag= and compare to extra field Tags *}
    {if (isset($entry->fieldsbyname.Tags->value)) AND (($entry->fieldsbyname.Tags->value|strpos:$tag) !== false)}
    {*... here do whole summary template as you want to have it ...*}
        <div class="CGBlogSummary">

Als ik Tag_CGBlog aanroep zoals ik dat doe om tags weer te geven, dan loopt de code gewoon door. Bovenin is juist $tag gedefinieerd om te voorkomen dat als deze template anders wordt aangeroepen, dat er dan geen foutmelding op strpos komt. Wat mij wel opvalt, is dat in de archive template 'Sample' wordt de template 'Jaar_Maand' aangeroepen, echter wordt die niet uitgevoerd. Heeft niets met de stap naar PHP 7 te maken, valt me alleen op.

Enig zoeken leert dat de manier waarop strpos wordt gebruikt tot meer vragen leidt. De code enigszins aangepast naar:

Code: Select all

{$containtag = {$entry->fieldsbyname.Tags->value|strpos:$tag}}
    {if (isset($entry->fieldsbyname.Tags->value)) AND (isset($containtag))}

hiermee geen foutmelding meer op de site, maar wel in de errorlog

[02-Apr-2017 19:26:41 UTC] PHP Warning: strpos(): Empty needle in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 59
[02-Apr-2017 19:26:41 UTC] PHP Notice: Undefined index: title in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 101
[02-Apr-2017 19:26:41 UTC] PHP Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 101
[02-Apr-2017 19:26:41 UTC] PHP Warning: strpos(): Empty needle in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 59

[Gregor]

Zie net dat:

Code: Select all

Code:
{$containtag = {$entry->fieldsbyname.Tags->value|strpos:$tag}}
    {if (isset($entry->fieldsbyname.Tags->value)) AND (isset($containtag))}

niet hetzelfde resultaat op de frontend heeft als deze code:

Code: Select all

    {if (isset($entry->fieldsbyname.Tags->value)) AND (($entry->fieldsbyname.Tags->value|strpos:$tag) !== false)}
    

maar dat het dit moet zijn:

Code: Select all

    {$containtag = {$entry->fieldsbyname.Tags->value|strpos:$tag}}
    {if isset($entry->fieldsbyname.Tags->value) AND !empty($containtag)}

In de errorlog levert dat nog wel de volgende foutmeldingen op:

[03-Apr-2017 04:40:01 UTC] PHP Notice: Undefined index: Tags in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 60
[03-Apr-2017 04:40:01 UTC] PHP Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 60
[03-Apr-2017 04:40:01 UTC] PHP Notice: Undefined index: Tags in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 60
[03-Apr-2017 04:40:01 UTC] PHP Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(165) : eval()'d code on line 60

De fout zit in deze regel code:

Code: Select all

    {$containtag = {$entry->fieldsbyname.Tags->value|strpos:$tag}}

Hij lijkt 'Tags' niet te herkennen???

[Rolf]

{$entry->fieldsbyname.Tags->value} heeft op dat punt geen waarde, dat *hoeft* geen fout te zijn.

Probeer het eens zo:

Code: Select all

{$tag = $smarty.get.tag|default:''}

{if !empty($tag)}
    <h5>Artikelen gerelateerd aan {$tag}</h5>

    {foreach from=$items item=entry}
        {if !empty($entry->fieldsbyname.Tags->value)}
            {if (($entry->fieldsbyname.Tags->value|strpos:$tag) !== false)}
                 ....
            {/if}
        {/if}
    {/foreach}
{/if}

[Gregor]

De if-statements heb ik op een iets andere plaats gezet waardoor beide overzichten werken, echter de foutmeldingen in de log die blijven.

De eerste meldingen zijn als er de archive template wordt aangeroepen en de tweede (laatste) melding als er via een tag een overzicht wordt getoond:

[03-Apr-2017 15:37:42 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 160
[03-Apr-2017 15:37:42 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 160
[03-Apr-2017 15:37:42 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 160
[03-Apr-2017 15:37:42 UTC] PHP Notice: Undefined index: year in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/dd06051ff96302c594f48d0d81e7b944959d9fd6.module_db_tpl.CGBlog;archiveSample.php on line 160

Onderstaande meldingen als er via een tag een artikel wordt geopend. Nu staat er in het artikel een youtube film en het meegegeven id wordt niet herkend.

[03-Apr-2017 15:38:06 UTC] PHP Notice: Undefined index: cgblog_id in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 67
[03-Apr-2017 15:42:52 UTC] PHP Notice: Undefined index: title in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 11
[03-Apr-2017 15:42:52 UTC] PHP Notice: Use of undefined constant id - assumed 'id' in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 16
[03-Apr-2017 15:44:23 UTC] PHP Notice: Undefined index: Tags in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/cgbd268^4e9782aecfeec71b029b4ec9d5a5b226e25863ce.module_db_tpl.CGBlog;detailtag_cgblog.php on line 80
[03-Apr-2017 15:44:23 UTC] PHP Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/cgbd268^4e9782aecfeec71b029b4ec9d5a5b226e25863ce.module_db_tpl.CGBlog;detailtag_cgblog.php on line 80
[03-Apr-2017 15:44:37 UTC] PHP Notice: Use of undefined constant id - assumed 'id' in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 16
[03-Apr-2017 15:44:52 UTC] PHP Notice: Undefined index: title in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 11
[03-Apr-2017 15:44:52 UTC] PHP Notice: Use of undefined constant id - assumed 'id' in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 16

De code van de UDT is:

Code: Select all

$id = 1;
$title = $params['title'];
if (empty($title)) {
    $title="Youtube video";
}

echo '<a href="http://youtu.be/'.$params[id].'" style="float:right;" class="floatbox" data-fb-options="addVideoThumb:192 caption2:`'.$title.'` width:592 height:333"></a>';

In het artikel verschijnt vervolgens de foutmelding:

Notice: Use of undefined constant id - assumed 'id' in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 17

Deze melding verschijnt alleen als PHP 7 actief is en niet bij PHP 5.6

[Rolf]

Code: Select all

$id = isset($params['id']) ? $params['id'] : '1';
$title = isset($params['title']) ? $params['title'] : 'Youtube video';

echo '<a href="http://youtu.be/'.$id.'" style="float:right;" class="floatbox" data-fb-options="addVideoThumb:192 caption2:`'.$title.'` width:592 height:333"></a>';

[Gregor]

Code: Select all

$id = isset($params['id']) ? $params['id'] : '1';
$title = isset($params['title']) ? $params['title'] : 'Youtube video';

echo '<a href="http://youtu.be/'.$id.'" style="float:right;" class="floatbox" data-fb-options="addVideoThumb:192 caption2:`'.$title.'` width:592 height:333"></a>';

[Gregor]

Bij het toevoegen van een artikel in CGBlog verschijnt bij het openen van de pagina bovenin de volgende foulmelding:

Notice: Undefined index: size in /........./modules/CGBlog/action.admin_addarticle.php on line 357

Zal kijken of dit als bug al is gemeld en anders zal ik 'm melden.

[velden]

Merk op dat het in principe aanbevolen wordt om in een productieomgeving de weergave van foutmeldingen uit te zetten.

Beter zorg je ervoor dat ze enkel in de log(file) verschijnen.

Verder is een notice géén foutmelding.