Fehler in Trunk 1075

[susigross]

Auf der Suchergebnisseite gibt es 2 Formulare, die beide mit

Code: [Select]

<form name="search" anfangen.
Namen sollten aber eindeutig sein.

Vieleicht kann man das ja noch in Ordnung bringen.

[ruebenwurzel]

Hallo,

ich habe auf der "Suchergebnisseite" im Quelltext immer nur ein Formular. Wieso hast du zwei?

Matthias

[susigross]

Ganz neue Installation, nichts geändert.
Auf der Suchergebnisseite hat man dann links das Suchformular, was immer zu sehen ist.
Außerdem hat man über den Suchergebnissen noch ein Formular mit der Erweiterung "Alle Wörter Einzelne Wörter Genaue Wortfolge"

Wieso hast du nur 1 Formular?

[susigross]

Kein richtiger Fehler, aber verbesserungswürdig (hat mich unter WB 2.7 mal einiges an Zeit gekostet):

in admin/pages/settings2.php
Zeile 171 aktuell:

Code: [Select]

if(!is_writable(WB_PATH.PAGES_DIRECTORY.'/')) {
Zeile 171 besser:

Code: [Select]

if(!is_writable(dirname($filename).'/')) {
Es ist nämlich wenig hilfreich, die Schreibrechte in einem anderen Verzeichnis zu testen als dort, wo man schreiben will.
Die Annahme, die Rechte wären identisch, trifft nicht immer zu, wie ich lernen musste.

Zeile 172 erweitert mann dann hilfreich so oder ähnlich:

Code: [Select]

$admin->print_error($MESSAGE['PAGES']['CANNOT_CREATE_ACCESS_FILE']." (FN: $filename)");

[Luisehahne]

@Matthias,

habe das jetzt im Roundtemplate ausprobiert. Links steh tdas Scuhefeld Bei erfolgloser Suche erscheint im Contont das erweiterte Suchfeld. Also ist das 2 mal vorhanden. macht aber keine Probleme und ist vaiide oder worum gehts?

Dietmar

[susigross]

@Matthias,

habe das jetzt im Roundtemplate ausprobiert. Links steh tdas Scuhefeld Bei erfolgloser Suche erscheint im Contont das erweiterte Suchfeld. Also ist das 2 mal vorhanden. macht aber keine Probleme und ist vaiide oder worum gehts?

Dietmar

1. Passiert das auch wenn die Suche erfolgreich ist.
2. ist es nicht valide

An anchor with the same name is already defined. Each anchor should be unique.

[Luisehahne]

Getestet, keine Probleme.

Dietmar

[susigross]

Getestet, keine Probleme.

Dietmar

So benimmt sich ein Developer? [EDIT: Persönlich Beleidigung entfernt.[/EDIT by kweitzel]

[Luisehahne]

Die Annahme, die Rechte wären identisch, trifft nicht immer zu, wie ich lernen musste.

Du meinst du hattest im Subverzeichnis der /pages/ andere Rechte? oder wo lag das Problem? Interessiert mich, so dass ich es nachverfolgen kann und mal austeste. In dem von dir beschriebenen Abschnitt geht es ja um das Umbenennen eines Title.

Obwohl die Rechte ja identisch mit der /pages/ sein

// Create a new file in the /pages dir if title changed

Dietmar

[ruebenwurzel]

Hallo,

bin dann später auch drauf gekommen, dass die zweite Suchmaske ja im Template liegt. Müßten also eigentlich nur die Templates angepasst werden (und die Hilfeseiten auf den erläutert wird, wie man eine suchmaske einbindet) wenn ich das richtig sehe.

in admin/pages/settings2.php
Zeile 171 aktuell:
Code:

if(!is_writable(WB_PATH.PAGES_DIREC TORY.'/')) {


Zeile 171 besser:
Code:

if(!is_writable(dirname($filename).'/')) {


Es ist nämlich wenig hilfreich, die Schreibrechte in einem anderen Verzeichnis zu testen als dort, wo man schreiben will.
Die Annahme, die Rechte wären identisch, trifft nicht immer zu, wie ich lernen musste.

Zeile 172 erweitert mann dann hilfreich so oder ähnlich:
Code:

$admin->print_error($MESSAGE['PAGES']['CANNOT_CREATE_ACCES S_FILE']." (FN: $filename)");

Hmm, muss ich auch mal reinschauen. Ich verstehe dich richtig, dass es besser wäre nicht den Ordner in dem die pages files liegen auf schreibberechtigung en zu testen, sondern nur die Datei, die man verändern will. Wenn ich dich so richtig verstanden habe macht das durchaus sinn.

Matthias

[susigross]

Hallo Matthias,

wenn doch alle Antworten in diesem Forum so sachlich wären wie deine

Zu den Schreibrechten:
Bisher wird getestet, ob man in der root Schreibrechte hat. Wenn das nicht der Fall ist, wird abgebrochen, auch wenn man gar nicht in die root schreiben will.
Es wird ja eine neue Datei angelegt (mittels create_access_file), also muß der Ordner, in den die Datei soll, das rechtemäßig erlauben.
Sicher ist es sinnvoll, daß das auch gleich in der Funktion create_access_file angepasst wird.

Zur Suchfunktion:
Wenn die zweite Suchmaske im Template liegt (wovon es ja nun ein paar hundert gibt), wäre es vielleicht einfacher, den Namen der ersten Suchmaske zu ändern?

[erpe0812]

wenn doch alle Antworten in diesem Forum so sachlich wären wie deine grin

Wer im Glashaus sitzt sollte nicht mit Steinen schmeissen:

So benimmt sich ein Developer? Oder ein Holzkopf?

Das geht für meinen Geschmack schon unter die Gürtellinie und gehört nicht hierher.


Gruss

erpe

[Luisehahne]

Hallo FRank,

Ich weiss nicht, warum du die Probleme hattest. Habe mal eben ein in ein paar WB Installationen nachgeschaut. In jedem Subverzeichnis sind die selben Rechte, wie im Rootverezichnis, halt die Einstellung unter erweiterte Optionen
im Backend. Deswegen war es bisher auch nicht nötig mehr als das Rootverzeichnis abzufragen. Oder sehe ich da was verkehrt?

Dietmar

[susigross]

Hallo FRank,

Ich weiss nicht, warum du die Probleme hattest. Habe mal eben ein in ein paar WB Installationen nachgeschaut. In jedem Subverzeichnis sind die selben Rechte, wie im Rootverezichnis, halt die Einstellung unter erweiterte Optionen
im Backend. Deswegen war es bisher auch nicht nötig mehr als das Rootverzeichnis abzufragen. Oder sehe ich da was verkehrt?

Dietmar

Man kann eine Installation ja auch sicherer machen, so daß nicht überall alles geändert werden kann.
Man muß ja auch kein pages-Verzeichnis haben, sondern kann gleich in der root anfangen und für die besondere Rechte vergeben.
Ich habe es aber satt, alles in langwierigen Diskussionen 3x begründen zu müssen, nur weil jemand die naheliegende allgemeingültige Lösung nicht akzeptieren will. Wahrscheinlich werde ich mir private Patchdateien anlegen und die dann ins fertige WB 2.8 übernehmen, das schont dann meine Nerven gewaltig.
Der Patch für die Mehrsprachigkeit von WB 2.8 ist gleich fertig

[Luisehahne]

Hallo Frank,

weist du ich denke wahrscheinlich weiter als du. Wenn sich die Rechte in einem Subverzeichnis ändern, dann liegt irgendwo ein Fehler vor, entweder ein menschlicher und von WB. Ich kann natürlich alles umschiffen, indem ich das Core an alle Unzugänglichkeiten anpasse. Klar macht die Fehlersuche natürlich viel Arbeit. Aber wenn die Rechte im Sub anders sind als in der /pages/ muss man sich doch als allererstes fragen, warum.

Man kann es sich natürlcih auch bequem machen. Ist aber nicht mein Ding und sollte auch nicht vom Dev-Team so praktiziert werden. Wir wollen natürlich ein fehlerfreies System, aber nicht auf die Art und Weise.

Dietmar

[susigross]

weist du ich denke wahrscheinlich weiter als du. Wenn sich die Rechte in einem Subverzeichnis ändern, dann liegt irgendwo ein Fehler vor, entweder ein menschlicher und von WB.

Ich habe keine Lust, über deine Vermutung zu diskutieren, behalte da mein Wissen lieber für mich.
Der Fehler besteht nicht darin, dass jemand manuell die Rechte ändert, sondern darin, dass WB danach nicht mehr logisch funktioniert, sprich dass man keine Seite mehr in einem Verzeichnis anlegen kann, für das man Schreibrechte hat.

[Luisehahne]

Genau da ist das Problem, du machst es dir zu einfach. Du willst das ja noch nicht mal ausdiskutieren, so dass wir auf einen Nenner kommen. Also hatte ich doch recht mit meiner Vermutung

Der Fehler besteht nicht darin, dass jemand manuell die Rechte ändert,

und WB soll das jetzt ausbügeln.

Könnte mich mit dem Vorschlag anfreunden, Vergleiche Rootrechte der /pages/ mit den Subs, wenn ungleich Warnung. Das wäre für mich die Lösung. So Ideen kommen dann eben bei einer Diskussion, auch wenn man manchmal aneinander vorbei redet.

Du kannst natürlich auch stur bleiben oder denkst über meinen Vorschlag nach. Das überlasse ich dir.

Wie war das mit dem Holzkopf? Lassen wir die FRage im Raume stehen.

Dietmar

[erpe0812]

So, jetzt ist ja wohl mehr als alles gesagt............. ...

Das sinnvollste ist doch wohl, wenn sich das DEV-Team dieser Geschichte annimmt,  es intern zu Ende diskutiert und dann entscheidet, was gemacht wird.
Wenn es noch Sachargumente gibt, die der Sache dienlich sind, dann kann das hier ja noch gepostet werden.

Ansonsten solltet ihr aufhören, euch hier zu beharken. Oftmals ist es besser, wenn dann mal ein bisher unbeteiligter aber erfahrener Dritter draufschaut, der weiss, worüber da gesprochen wird.

In diesem Sinne einen schönen Tag noch.

Gruss

erpe

[Luisehahne]

Hallo Frank,

auch wenn du nicht mehr antworten willst, muss ich dir Recht geben, das da Handlungsbedarf besteht. Habs gerade mal durchgetestet und ein paar Rechte verändert. WB sagt ist alles ok, legt aber nichts an und das ohne Fehlermeldung. Werde da mal weiter verfolgen.

Trotzdem danke für die Anregung.

Dietmar

[susigross]

Hier ist noch ein Bug, Diskussionen darüber bitte ohne mich:
https://forum.WebsiteBaker.org/index.php/topic,13754.msg91513.html#msg91513

[susigross]

Noch etwas unschönes was die Usability betrifft:
Man legt einen neuen Benutzer an, klickt auf Hinzufügen und bekommt die Meldung, das Passwort wäre zu kurz.
Schlecht dabei

• Man erfährt nicht, wie lang das PW sein muss
• alle anderen Eingaben zum neuen Benutzer werden gelöscht, man muss also von vorne anfangen

[Luisehahne]

Hallo Frank,

Stimmt, dass hatte mich auch schon gestört und habe es auf meine Feature List gesetzt, dass inputs direkt bei der Eingabe zu validiert werden. Werde dies für die WB 2.9 anregen.

Dietmar

[susigross]

Hallo Frank,

Stimmt, dass hatte mich auch schon gestört und habe es auf meine Feature List gesetzt, dass inputs direkt bei der Eingabe zu validiert werden. Werde dies für die WB 2.9 anregen.

Dietmar

Schade, der Bug, dass alle Felder gelöscht werden, wurde ja erst in 2.8 eingebaut. Da hätte ich erwartet, daß er bis zum endgültigen Release auch wieder ausgebaut wird, und nicht erst in 2.9

[ruebenwurzel]

Hallo,

Schade, der Bug, dass alle Felder gelöscht werden, wurde ja erst in 2.8 eingebaut. Da hätte ich erwartet, daß er bis zum endgültigen Release auch wieder ausgebaut wird, und nicht erst in 2.9

Sicher, dass das nicht auch in der 2.7 schon so war? Kannst du das auch nochmal checken?

Matthias

[erpe0812]

Hallo

bin mir ziemlich sicher, dass das schon immer so war.
Habe mich auch schon öfter darüber geärgert.

Gruss

erpe