Notice bei RT Last update artikel & cp

Bitte alle templatebezogenen Beiträge in diesem Forum veröffentlichen.
Post Reply
User avatar
Uwe367
Posts: 974
Joined: Sat 15. Sep 2007, 07:19
Location: Koblenz

Notice bei RT Last update artikel & cp

Post by Uwe367 » Sun 12. Oct 2014, 15:51

Hallo zusammen,
Dieses Script aus dem Wiki
http://www.phpwcms-howto.de/wiki/doku.p ... s[]=update
gab mir eine Notice aus.

Code: Select all

Notice: Undefined index: acat_archive in E:\xampp\htdocs\newsite\template\inc_script\frontend_render\rt_date_last_update.php on line 85
Ok.. ne Notice ist zwar noch kein Fehler, aber man sollte sie nicht mißachten.
Die Zeile sah im Original so aus:

Code: Select all

if ($content["struct"]['0']['acat_permit']['acat_archive'] == 1)
und ich habe das nun folgendermaßen gefixt:

Code: Select all

if (isset($content["struct"]['0']['acat_permit']['acat_archive']) == 1)
Die Notice ist weg und das Script arbeitet einwandfrei.
Ist das so ok?

User avatar
Oliver Georgi
Site Admin
Posts: 9552
Joined: Fri 3. Oct 2003, 22:22
Location: Moscow, Russia
Contact:

Re: Notice bei RT Last update artikel & cp

Post by Oliver Georgi » Sun 12. Oct 2014, 17:23

Nein, ist grundsätzlich falsch. Es gibt keinen solchen Wert.

Entweder $content["struct"][0]['acat_permit'] oder $content["struct"][0]['acat_archive'].

Code: Select all

if(!empty($content["struct"][0]['acat_archive']))
Oliver Georgi | phpwcms Developer | GitHub | LinkedIn | Систрон

User avatar
Uwe367
Posts: 974
Joined: Sat 15. Sep 2007, 07:19
Location: Koblenz

Re: Notice bei RT Last update artikel & cp

Post by Uwe367 » Sun 12. Oct 2014, 18:43

Danke Oliver :)

User avatar
juergen
Moderator
Posts: 4553
Joined: Mon 10. Jan 2005, 18:10
Location: Weinheim
Contact:

Re: Notice bei RT Last update artikel & cp

Post by juergen » Mon 13. Oct 2014, 09:45

Das ist der lachse Umgang von PHP mit Variablentypen... string und integer

Der Link zum Wiki ist auch von forum zerfetzt, hat das jemand mal geändert ?

User avatar
Oliver Georgi
Site Admin
Posts: 9552
Joined: Fri 3. Oct 2003, 22:22
Location: Moscow, Russia
Contact:

Re: Notice bei RT Last update artikel & cp

Post by Oliver Georgi » Mon 13. Oct 2014, 10:05

juergen wrote:Das ist der laxe Umgang von PHP mit Variablentypen... string und integer
Nein, das ist wirklich falsch und hat nichts mit den Variablentypen zu tun. Die meisten Anwender nehmen es nur nicht wahr, da sie zu laxe Fehlerreporting-Einstellungen verwenden und so eben NULL Fehler sehen. Jede Notice oder Warning sollte ausgemerzt werden. Entwicklung also immer nur mit E_ALL besser E_STRICT, ^E_NOTICE und ^E_DEPRICATED meiden.
Oliver Georgi | phpwcms Developer | GitHub | LinkedIn | Систрон

User avatar
Uwe367
Posts: 974
Joined: Sat 15. Sep 2007, 07:19
Location: Koblenz

Re: Notice bei RT Last update artikel & cp

Post by Uwe367 » Mon 13. Oct 2014, 11:39

Kleines OT....

Ich arbeite mit xampp. Dieser verfügt über 2 php.ini Dateien, zum einen über php.ini-developement, zum anderen über die php.ini-production.
Bei beiden waren folgende Standartwerte gesetzt:

Code: Select all

; error_reporting
;   Default Value: E_ALL & ~E_NOTICE & ~E_STRICT & ~E_DEPRECATED
;   Development Value: E_ALL
;   Production Value: E_ALL & ~E_DEPRECATED & ~E_STRICT
Wenn ich das nun richtig verstehe wäre es besser diese auf

Code: Select all

; error_reporting
;   Default Value: E_ALL & ~E_NOTICE & ~E_STRICT & ~E_DEPRECATED
;   Development Value: E_STRICT
;   Production Value: E_ALL & ~E_DEPRECATED & ~E_STRICT
zu ändern?

Post Reply