CMS Made Simple Forums

Content management as it is meant to be
https://forum.cmsmadesimple.org/

Lay-out iPad php versie 7

https://forum.cmsmadesimple.org/viewtopic.php?t=76404

Page 2 of 5

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 12:06 pm
by velden
Het laden van jQuery is niet aangepast tussen de verschillende php-versies. Dat zou dus betekenen dat php7 de template anders processed?
Nee, dat betekent dat het daarvoor ook al stuk was :-)

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 12:07 pm
by Gregor
Fijn... ;)

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 12:34 pm
by Rolf
http://viewsvn.cmsmadesimple.org/diff.p ... module.php&

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 1:04 pm
by Gregor
Die werkt ;)

Blijven er nog een aantal fouten zitten die, denk ik, met syntax-verschillen te maken hebben. Kan dat niet inschatten.

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 2:46 pm
by Gregor
Kan vorige post niet meer editten, daarom zo.

Wat ik nu nog zie op blogpagina's:
Nog geen reacties | Dit artikel is 327 keer bekeken.

Notice: Undefined index: return_url in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(151) : eval()'d code on line 71

Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(151) : eval()'d code on line 71
Tags
Notice: Undefined index: stats in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(151) : eval()'d code on line 100

Notice: Trying to get property of non-object in /home/efacti/public_html/uisge-beatha.eu/www/lib/smarty/sysplugins/smarty_internal_templatebase.php(151) : eval()'d code on line 100
Op dit artikel zijn nog geen reacties... Laat als eerste een reactie achter!
Artikelen binnen dezelfde categorie

Notice: Undefined index: pageid in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 6

Notice: Undefined index: currid in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 7
De bijbehorende code is van de detail cgblog pagina is:

Code: Select all

{strip}
{* get_template_vars *}
{* +++++ Start SEO tags +++++ *}
{$canonical = $entry->canonical scope=global}
{$last_modified = $entry->modified_date|date_format:'%e-%m-%Y' scope=global}
{if !empty($entry->fieldsbyname.facebook_thumbnail->value)}
    {$page_image = "{$entry->file_location}/{$entry->fieldsbyname.facebook_thumbnail->value}" scope=global}
{/if}
{$page_title = $entry->title|cms_escape:htmlall scope=global}
{* +++++ End SEO tags +++++ *}

{assign var='detail_page' value='logboek'}
{assign var='uisge_news_id' value=$entry->id}

{if $entry->category}
	<div id="NewsPostDetailCategory">
		{$category_label} {$entry->category}
	</div>
{/if}

{if $entry->postdate}
   <div id="NewsPostDetailDate">
      {$entry->postdate|date_format:'%a, %d %b %Y'}
   </div>
{/if}


{if $entry->summary}
   <div id="NewsPostDetailSummary">
	{eval var=$entry->summary}
   </div>
{/if}


<div id="NewsPostDetailContent">
        {eval var=$entry->content}
</div>

<div id="BlogCounter">
   {CGFeedback key1='CGBlog' key2=$entry->id action='ratings' ratingstemplate='feedback_counter'}  |  Dit artikel is {HitCounter key1="CGBlog" key2=$entry->id} keer bekeken.
</div>


{if $return_url != ""}
  <div id="NewsPostDetailReturnLink">{$return_url}</div>
{/if}


{if $entry->category == 'Logboek' || $entry->category == 'Tochten' ||  $entry->category == 'Onderhoud'|| $entry->category == 'Hardlopen' ||  $entry->category == 'General' || $entry->category == 'Statistiek'}
categorie </br>
  {cms_module module='comments' modulename='News' pageid=$entry->id}
categorie 1<br />
  {CGFeedback key1="News" key2=$entry->id lang="nl_NL"}
  key2 ja/nee: ({$entry->id|print_r}) <br />
  {CGFeedback key1="CGBlog" key2=$entry->id  lang="nl_NL"} 
  {CGFeedback key1="CGBlog" key2=$entry->id action='summary' lang="nl_NL"} 
  uisge_news_id ja/nee: ({$uisge_news_id})
categorie 2<br />
  {CGFeedback key1="CGBlog" key2=$uisge_news_id  lang="nl_NL"} 
  {CGFeedback key1="CGBlog" key2=$uisge_news_id action='summary' lang="nl_NL"}
{/if}
{if isset($entry->fields)}
    {assign var="taglist" value=$entry->fieldsbyname.Tags->value}
    {assign var="tagwords" value=", "|explode:$taglist}
      {foreach from=$entry->fields item='field'}
            {* <div class="CGBlogDetailField"> *}
                {if $field->type == 'image'}
                    <img class="hidden" src="{$entry->file_location}&#47;{$field->value}" alt="{$title}" />
{* $tst_ogtitle    27oct16: class hidden doesn't work in Safari *}
                {elseif $field->name == 'Tags'}
                    {if $field->name != ""} {* tags to show *}
                           {* $field->name} >> *}
                           <h5>Tags</h5>
                           {* assign var="taglist" value=$entry->fieldsbyname.Tags->value *}
                           {* taglist :$taglist|print_r *}
                           {*assign var="tagwords" value=", "|explode:$taglist*}
                           <ul id="taglist">
                                {foreach from=$tagwords item=tagword}
                                {if $tagword != ''}
                                    <li>
                                        <a href="{root_url}/logboek&tag={$tagword}"><span class="fa fa-tag"></span>&nbsp;&nbsp;{$tagword}</a>
                                        {* cms_selflink page='logboek' urlparam="&tag=`$tagword`" text=$tagword|trim*} {* cgblog_logboek *}
                                    </li>
                                {/if}
                                {/foreach}
                           </ul>
                           <div style="clear:both"></div>
                            <br />
                    {/if}
                {else}
                      <p>{$field->name}: {eval var=$field->value}</p>
                {/if} 
            {*</div>*}
      {/foreach}
{/if}

<h5>Reacties</h5>

<div class="fb-comments" data-href="{$canonical}" data-num-posts="5" data-width="470"></div>

{if $stats.count !=0}
    {CGFeedback key1="CGBlog" key2=$uisge_news_id action="summary" lang="nl_NL"}
    {form_ip_filter}
    {if empty($ipbanned) || $ipbanned != 'true'}
      {startExpandCollapse id="Reageeropditartikel" title="Klik om ook op dit artikel ook te reageren."}
        {CGFeedback key1="CGBlog" key2=$uisge_news_id action="default" lang="nl_NL"}
      {stopExpandCollapse}
    {/if}
    {startExpandCollapse id="Reageeropditartikel" title="Klik om ook op dit artikel ook te reageren."}
        {CGFeedback key1="CGBlog" key2=$uisge_news_id action="default" lang="nl_NL"}
    {stopExpandCollapse}
{else}
   {form_ip_filter}
   {if empty($ipbanned) || $ipbanned != 'true'}
      <p>Op dit artikel zijn nog geen reacties... Laat als eerste een reactie achter!</p>
      {startExpandCollapse id="Reageeropditartikel" title="Klik om op dit artikel te reageren."}
        {CGFeedback key1="CGBlog" key2=$uisge_news_id action="default" lang="nl_NL"}
      {stopExpandCollapse}
   {/if}
{/if}
<br /><br />

<h5>Artikelen binnen dezelfde categorie</h5>
{prev_next pageid=$page_id currid=$entry->id currdate=$entry->postdate}
<div class="prevnext">
   {if $cgblog_next_url}<a href="{$cgblog_next_url}"><< Vorig artikel in deze categorie</a> | {/if}
   {if $cgblog_prev_url}<a href="{$cgblog_prev_url}">Volgend artikel in deze categorie >></a>{/if }
</div>
<br />
{/strip}
Iemand die richting kan geven aan een oplossing?

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 3:44 pm
by Rolf

Code: Select all

{if $return_url != ""}

Code: Select all

{if !empty($return_url)}

Code: Select all

{if $stats.count !=0}

Code: Select all

{if !empty($stats.count)}

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 3:49 pm
by Rolf
Notice: Undefined index: pageid in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 6

Notice: Undefined index: currid in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 7
viewtopic.php?p=332559#p332559

Code: Select all

$pageid = ($_REQUEST['pageid']) ? $_REQUEST['pageid'] : $params['pageid'];
$currid = ($_REQUEST['currid']) ? $_REQUEST['currid'] : $params['currid'];
Je zult in de aanroep van deze UDT de twee parameters moeten meegeven. De requests zijn blijkbaar leeg.

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 4:08 pm
by Gregor
Er zijn met je post enkele foutmelding opgelost :)
Rolf wrote:
Notice: Undefined index: pageid in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 6

Notice: Undefined index: currid in /home/efacti/public_html/uisge-beatha.eu/www/lib/classes/class.usertagoperations.inc.php(278) : eval()'d code on line 7
viewtopic.php?p=332559#p332559

Code: Select all

$pageid = ($_REQUEST['pageid']) ? $_REQUEST['pageid'] : $params['pageid'];
$currid = ($_REQUEST['currid']) ? $_REQUEST['currid'] : $params['currid'];
Je zult in de aanroep van deze UDT de twee parameters moeten meegeven. De requests zijn blijkbaar leeg.
Doe ik dat hier dan niet?

Code: Select all

{prev_next pageid=$page_id currid=$entry->id currdate=$entry->postdate}

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 4:24 pm
by Rolf
Gregor wrote:Doe ik dat hier dan niet?

Code: Select all

{prev_next pageid=$page_id currid=$entry->id currdate=$entry->postdate}
Theoretisch wel, maar als op dat punt de strings geen waarde hebben, dan doe je het niet...

Zou je controleren met bijv.:

Code: Select all

-{$page_id}-<br />
-{$entry->id}-<br />
{prev_next pageid=$page_id currid=$entry->id currdate=$entry->postdate}
Maar als alles verder wel werkt en je hebt geen zin om verder te puzzelen, kun je de melding waarschijnlijk ook wel omzeilen door bovenin de UDT te plaatsen:

Code: Select all

$pageid = '';
$currid = '';

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 5:02 pm
by Gregor
Testje gedaan en zie hier:

Code: Select all

-{$page_id}-<br />
-{$entry->id}-<br />
-30-
-480-
Parameters zijn er wel en worden ook in de UDT ontvangen:
30480
Vermoed dat de "fout" in dit stukje code zit (de twee echo-statements zijn ter test:

Code: Select all

$gCms = cmsms();
$db = cmsms()->GetDb();    // changed this line.
$cgblog = cms_utils::get_module('CGBlog');  // changed this line.

// call from cgblog detail_template: {prev_next pageid=$page_id currid=$entry->id currdate=$entry->postdate}

$pageid = ($_REQUEST['pageid']) ? $_REQUEST['pageid'] : $params['pageid'];
$currid = ($_REQUEST['currid']) ? $_REQUEST['currid'] : $params['currid'];  // added this line.
$currdate = $params['currdate'];
if(!$currdate) return;

echo $pageid;
echo $currid;

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 5:12 pm
by Rolf
Als de parameters altijd via de tag werken, dan kun je beter zoiets doen:

Code: Select all

$pageid = isset($params['pageid']) ? $params['pageid'] : '';
$currid = isset($params['currid']) ? $params['currid'] : '';
ipv

Code: Select all

$pageid = ($_REQUEST['pageid']) ? $_REQUEST['pageid'] : $params['pageid'];
$currid = ($_REQUEST['currid']) ? $_REQUEST['currid'] : $params['currid'];

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 5:17 pm
by Gregor
Zat net iets te lezen over de verandering van gebruik van $_REQUEST en de instelling in php.ini en dat daar mogelijk een oorzaak zit.

Je oplossing werkt Rolf :)

Dat kost best wat inspanning om PHP 7 aan de praat te krijgen....

Re: Lay-out iPad php versie 7

Posted: Wed Mar 29, 2017 9:58 pm
by Gregor
Ben er nog niet helemaal... Bij het selecteren van een tag onderaan een artikel, zou je verwachten dat de bijbehorende artikelen vanuit de CGBlog summary template worden weergegeven. De pagina laat daarnaast ook een hoeveelheid gelijke fouten zien:
Artikelen gerelateerd aan Raymarine
Draaiende windmeter
LOGBOEK DATUM: 10 dec 2016
Vastzittende lagers in Raymarine ST60 masttopunit.
Linear drive stop
LOGBOEK DATUM: 10 jul 2016
Herstellen van de melding "Drive stop" van de Raytheon type 1 stuurautomaat.
Impuls aankoop
LOGBOEK DATUM: 06 jun 2015
Een weekje zeilen gaat een verbetering op mijn eigen boot geven...

Notice: Undefined index: Tags in /home/efacti/public_html/uisge-beatha.eu/www/tmp/templates_c/9488c580b6cd865d519661bb4d58f626cf3f30d0.module_db_tpl.CGBlog;summaryTag_CGBlog.php on line 54

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 54
De code waar het mis gaat is hier:

Code: Select all

{if (($entry->fieldsbyname.Tags->value|strpos:$tag) !== false)}
Dat levert met PHP 7 dezelfde fouten op en daarbij ontbreekt er 1 artikel, in dit geval althans. Heb nog op de cmscanbesimple-site gekeken, maar daar staat dezelfde code. Wat zou er onjuist aan de code zijn?

Re: Lay-out iPad php versie 7

Posted: Thu Mar 30, 2017 7:40 am
by velden
Notices zijn geen errors. Desalniettemin gaat er iets niet helemaal goed.

In dit geval bestaat '$entry->fieldsbyname.Tags' niet. Waarom kan ik niet beoordelen. Is het misschien zo dat in de loop van de tijd er iets veranderd is in CGBlog maar jij nog op een 'oude' manier met je templates werkt? Je zou eens kunnen kijken in de CGBlog help of een voorbeeld template om te zien hoe je die custom fields moet oproepen in je template.

Code: Select all

<pre>{$entry|print_r}</pre>
geeft misschien ook wel duidelijkheid.

Re: Lay-out iPad php versie 7

Posted: Thu Mar 30, 2017 9:42 am
by Rolf

Code: Select all

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

Code: Select all

{if (isset($entry->fieldsbyname.Tags->value)) AND (($entry->fieldsbyname.Tags->value|strpos:$tag) !== false)}
All times are UTC
Page 2 of 5

Powered by phpBB® Forum Software © phpBB Limited