gelöst - [browser_update] nach Installation kein Zugriff mehr auf Seitenlayouts

[abackhaus]

Hallo,

nach der Installation von browser_update kann ich die Seitenlayouts im Backend nicht mehr aufrufen:

Est kommt ne FF - Fehlermeldung:

Content-Encoding-Fehler
Die Webseite, die Sie öffnen möchten, kann nicht angezeigt werden, da sie ein ungültige oder unbekannte Form der Kompression verwendet.

und im Log:

PHP Fatal error: Uncaught exception 'Exception' with message 'Query error: Duplicate column name 'browserUpdateEnabled' (ALTER TABLE `tl_layout` ADD `browserUpdateEnabled` varchar(1) NOT NULL default ''' thrown in /srv/www/vhosts/xxxxxxxx.xxx/subdomains/xxx/httpdocs/system/libraries/Database.php on line 533

und jede Menge Warnungen:

PHP Warning: Call-time pass-by-reference has been deprecated in /srv/www/vhosts/Xxxxxxxx.xxx/subdomains/xxxxx/httpdocs/system/modules/browser_update/dca/tl_layout.php on line xxxx

wenn ich die Extension entferne ist alles wieder ok.

Hat jemand 'ne Idee ?

[Toflar]

Etwas stimmt mit der DB nicht...

Hast du mal versucht, nach dem Löschen der Dateien, deine DB upzudaten (entweder via Install-Tool oder Erweiterungsverwaltung) und das Modul danach frisch zu installlieren?

[abackhaus]

Hallo

die DB funktioniert wunderbar und ist aktuell. Ich hab die Erweiterung mittlerweile nur im backend/Einstellungen deaktiviert. Dann geht alles wie geschmiert und ohne Fehlermeldungen. Sobald die Erweiterung wieder aktiviert ist geht's nicht mehr. Hab auch shon ein paar mal deinstalliert und frisch installiert - nützt nichts

[Toflar]

Jau, dann gibt's nur zwei mögliche Ursachen:

• Das Modul funktioniert nicht korrekt (wobei das komisch ist, weil du bist ja nicht der einzige der es nutzt und bis anhin bis du der erste, der Probleme bekundet)
• Das Modul beisst sich mit einem anderen Modul

Woher kommt also deiner Meinung nach die Meldung

Duplicate column name 'browserUpdateEnabled' (ALTER TABLE `tl_layout` ADD `browserUpdateEnabled`

wenn du der Meinung bist, die DB sei sauber?

[abackhaus]

hm... ich habe ein sauber funktionierendes System und es geht nicht mehr, wenn ich eine Erweiterung installiere. Es geht dann wieder, wenn ich diese Erweiterung deinstalliere .... Das ist perfekt reproduzierbar.

Mein Rückschluss ist dann, das die Erweiterung ein Problem verursacht, zumal sie für Fehler und Warnungen im Log sorgt. Die Erfahrung zeigt auch, dass selbst wenn andere die Erweiterung benutzen, es evtl. doch noch Fehler geben könnte ....

Woher kommt also deiner Meinung nach die Meldung
Zitat:
Duplicate column name 'browserUpdateEnabled' (ALTER TABLE `tl_layout` ADD `browserUpdateEnabled`
wenn du der Meinung bist, die DB sei sauber?

na ja - wenn ich das mal wüsste, habe das Teil ja nicht programmiert.

Tatsache ist, die Erweiterung legt einige Spalten in der Tabelle tl_layout an. Wenn man in die DB schaut, so sind sie nach der Installation ordentlich darin. Wenn man die Erweiterung deinstalliert und die Spalten entfernt, so sind Sie nach einem prüfenden Blick auf die DB dann auch wieder weg. So sollte es doch wohl auch sein ?

Wie kommst Du denn auf die Idee, die Datenbank sei nicht in Ordnung ?

[Toflar]

Wie ich auf die Idee komme?

Na weil da "doppelter Eintrag" steht und bis jetzt niemand ein Problem damit hatte
Also gehe ich davon aus, dass es sich um etwas lokales handelt.

Ich sag ja nicht deine DB ist schuld - ich such nur nach Ansätzen
Hmm welche Erweiterungen hast du sonst noch am Laufen?

Mal schauen ob ich hier im Büro mit dem Standard-Notepad etwas im Code ausfindig machen kann -.-

[abackhaus]

Ich hab natürlich ne ganze Reihe von Erweiterungen im Einsatz:

Ajax , Abhängiges Formular , ab_cat2efg, Auflistung , backend_filterwiz, backend_tabletree, Benutzerregistrierung, browser_update, Catalog, catalogajaxratingfield, catalogitemhitsfield, catalogitemuseridfield, catalogmetafield, catalogmultitextwizardfield, ce_flash, clearlog, dfGallery
Entwickler-Tools, efg, Events, FAQ, Formular-Daten, Formularfeld "Fieldset", Formulargenerator Hilfsnachricht, Glossar, header_code, Kalenderfeld, Kommentare, licensewidget, Länder Select-Menü, Mitgliederliste, MultiTextWizard, Nachrichten, Newsletter, Repository-Base, Repository-Client, RSS-Reader, Sprachenwechsler, Standard-Wert leeren, Taxonomy, Templates, Weiterleitungs-Module , xtmembers_fieldeditor

[Toflar]

Okay, ich nutze das Modul selber nicht, deshalb meine Annahmen nur rein vom Code her:

Das Modul wird scheinbar mit der Aktivierung einer Checkbox im Seitenlayout aktiviert.
Dann sollte dieser Wert in der DB gespeichert werden:

`browserUpdateEnabled` varchar(1) NOT NULL default '',

nur sollten Checkboxen nicht so, sondern so erfasst werden:

`browserUpdateEnabled` char(1) NOT NULL default '',

Versuch mal das in der config/database.sql zu ändern und dann den Installationsvorgang zu wiederholen!

[abackhaus]

Hab den datentyp in der db von varchar auf char gesetzt. Hilft leider nicht weiter.

Trotzdem Danke für die Mühe

[Toflar]

Kontaktier mal Torben

[BugBuster]

Also das Duplicate... kann nur kommen, wenn mindestens 2 Module ein gleichnamiges Feld definieren.
Das kann ich mir allerdings bei dem Namen "browserUpdateEnabled" kaum vorstellen.

Die "Call-time pass-by-reference..." Fehler kann ich nach Sicht der Source auch nicht erklären, da keine Methoden mit einer Referenze als Parameter aufgerufen werden.
Hier wäre mal aus einer der Meldungen die Zeile interessant.
Ich hoffe das trifft nich einer der "&$GLOBALS['TL_LANG']...." Zeilen, dann wird es unheimlich.

Äh, nochwas: welche PHP Version ?

[abackhaus]

Ich verwende PHP Version 5.2.11 und es handelt sich bei der deprecated-Warnung um folgende Zeilen:

91,93,94,95 ,105,107,108,109,110,120,122,123,124,125,135,137,1 38,139,140,141

[BugBuster]

Hab mal ein wenig gesucht im Netz, PHP 5.2.11 mach auch woanders Probleme. Arbeitest Du unter Windows? (WAMP / XAMP o.ä)

[abackhaus]

Arbeitest Du unter Windows? (WAMP / XAMP o.ä)

nein - Linux Server

Users advised to update to PHP 5.2.11

PHP logo The developers of the PHP programming language are advising developers to update PHP 5.2.x-based applications to the new stable version 5.2.11. The update focuses on stability and contains more than 75 bug fixes, some security related.

The development team have fixed certificate validation inside php_openssl_apply_verification_policy and quality assurance for the colour index in imagecolortransparent(). They have also added a security check when processing EXIF (Exchangeable Image File Format) files. Further information can be found in the Release Notes for the new version.

[BugBuster]

Hab ich auch gelesen. Und ebend genausoft das diese und die 5.3.0 Probleme macht, wie hier um Forum auch schon öfters diskutiert, da diese stärkere Restriktionen haben und lange abgekündigte "Features" nun anmeckern.

2 Möglichkeiten erstmal damit weiterzuleben wäre in diesem Fall, die call-time-pass-reference zu erlauben.

Variante 1: in .htaccess

php_flag allow_call_time_pass_reference 1

Variante 2: in php.ini

allow_call_time_pass_reference = On

Hier sollte man vorher suchen ob es nicht dort explizit ausgeschaltet wird, dann dort von Off auf On ändern.

Was ich nur nicht verstehe ist, die Referenzen bei Methodenaufrufen werden nicht mehr unterstützt (aus Sicherheitsgründen, soweit ok), aber warum dann die Meldung bei Zuweisung eines Textes aus einer referenzierten Variablen?

Heisst aber auch, das Problem müsste dann fast alle Module treffen.

[abackhaus]

Hallo BugBuster,

danke, der Eintrag in die php.ini hat's gelöst - jetzt gehts.

Hab ich auch gelesen. Und ebend genausoft das diese und die 5.3.0 Probleme macht, wie hier um Forum auch schon öfters diskutiert, da diese stärkere Restriktionen haben und lange abgekündigte "Features" nun anmeckern.

Noch einige Anmerkungen: Es machten nicht, wie vermutet alle Erweiterungen (ich spreche hier mal von denen, die ich installiert habe) Problem - nur diese. Das geht klar aus dem log hervor.

Ich finde es wichtig, die PHP Versionen von Zeit zu Zeit auf den neuesten Stand zu bringen - schliesslich gibt es Sicherheitslücken und andere Fehler die beseitigt wurden. Die Systemvoraussetzungen für TL sprechen hier ja auch für PHP mindestens in der Version 5.2 - da gehe ich dann mal davon aus, dass die höheren Versionen - zumindest wenn sie schon eine Weile auf dem Markt sind auch unterstützt werden.

[BugBuster]

Noch einige Anmerkungen: Es machten nicht, wie vermutet alle Erweiterungen (ich spreche hier mal von denen, die ich installiert habe) Problem - nur diese. Das geht klar aus dem log hervor

Das macht die Sache noch unverständlicher für mich.
Ich dachte dabei daran, das TL die Module alphabetisch einliest, und da ist dieses nun mal ziemlich am Anfang.

[BugBuster]

So.
Nun habe ich mal, um das ganze nachzuvollziehen, WAMP Installiert, dort ist PHP 5.3.0 mit dabei.
Nun sah ich auch die Meldungen "PHP Warning: Call-time pass-by-reference has been deprecated ...", getestet mit Modul in Version 1.0.0 stable Build 6.
(in der php.ini is per default allow_call_time_pass_reference = Off)

Falls das einschalten wie von mir beschrieben nicht möglich ist oder nicht gewollt ist, kann man als Alternative auch die Datei /browser_update/dca/tl_layout.php editieren und dort alle

sprintf(&$GLOBALS[.... 

durch

sprintf($GLOBALS[.... 

ersetzen. (das & entfernen)
Hier wurde genau das getan was nun ab PHP 5.2.11 / 5.3.0 deprecated ist, eine Referenz als Funktionsparameter zu nutzen.

.