WBCE CMS Forum

colinax

Developer

[Erledigt] [1.2 Beta] Reload Probleme

Da es ja reload Probleme gibt, bin ich mal auf die Suche gegangen und habe das Upgradescript und die reload.php verglichen.

Dabei sind mir folgende Sachen aufgefallen:

1.
Während im Upgradescript einheitlich "and" verwendet wird, werden in der reload.php "AND" und "&&" verwendet.

On "and" groß oder klein geschrieben wird dürfte wurscht sein, aber bei "&&" kommt ja technisch was anderes heraus.

Welcher dieser Operatoren sollte verwendet werden?

2.
Welche dieser schreibweise ist die korrekte (von der logik her vermute ich mal A):

A:

if ($handle = opendir(WB_PATH . '/languages/')) {

oder B:

if ($handle = opendir(WB_PATH . '/templates')) {

bernd

Core-Team

Re: [Erledigt] [1.2 Beta] Reload Probleme

Ich denke mal (kann natürlich auch sein, das ich Bullshit erzähle),
zu 1.
AND  / OR (egal ob groß oder klein) und && / ||  dürfte in unseren Anwendungsfällen unerheblich sein.
AND und OR haben zwar eine niedrigere Rangfolge als && und || ich kann mir aber nicht vorstellen das das in den Abfrageb eine Rolle spielt.
zu 2.
da hab ich 'ne Weile gebraucht um zu kapieren auf was du rauswillst ;-)
Dir geht es um den "trailing slash".
Von der Logik her (und für die bessere "Lesbarkeit") schon Variante A, da ist sicher, das es sich um ein Verzeichnis handelt. In Variante B könnte "templates" auch eine Datei sein - in wie weit das im Zusammenhang mit opendir eine Rolle spielt ???

Beitrag geändert von bernd (12.05.2017 23:20:43)

---

... nein in Europa verwenden wir beim Programmieren nicht € statt $ ...

colinax

Developer

Re: [Erledigt] [1.2 Beta] Reload Probleme

Danke, für die Info.

Dir geht es um den "trailing slash".
Von der Logik her (und für die bessere "Lesbarkeit") schon Variante A, da ist sicher, das es sich um ein Verzeichnis handelt. In Variante B könnte "templates" auch eine Datei sein - in wie weit das im Zusammenhang mit opendir eine Rolle spielt ???

Gar keine, das sind nur mal schnell kopierte Inhalte aus der reload.php

Es kommt gleich ein Commit der aus den B's ein A draus macht.

Dann sollten die seltsamen Bugs beim reload behoben sein.

colinax

Developer

Re: [Erledigt] [1.2 Beta] Reload Probleme

Done.

[Erledigt]

grindbatzn

Gast

Re: [Erledigt] [1.2 Beta] Reload Probleme

@colinax
Das hier gesehen?
https://forum.wbce.org/viewtopic.php?pid=11693#p11693

Beitrag geändert von grindbatzn (13.05.2017 09:34:47)