Groß- Kleinschreibung bei Zugriffsdateinamen

[ruebenwurzel]

Hallo,

mir ist gerade folgendes aufgefallen:

- beim Anlegen neuer Seiten wird der Acessfile Name kleingeschrieben. Beim Seitenname "Test" wird eine "test.php" erstellt.
- wenn ich bei den Seitenoptionen (pages/settings.php) den Namen der Zugriffsdatei herauslösche und über speichern neu generiere, wird beim Seitenname "Test" eine "Test.php" erstellt.

Ich würde eine einheitliche Schreibweise der Zugriffsdateinamen bevorzugen. Also entweder alles kleingeschrieben (meine bevorzugte Variante) oder in der gleichen Schreibweise wie der Seitenname. Auf die Funktionalität hat beides keinerlei Einfluss, insofern besteht technisch gesehen kein zwingender Grund hier was zu ändern, mir persönlich gefällt aber der Mischmasch aus unterschiedlich geschriebenen Zugriffsdateinamen nicht.

Gruß
Matthias

Ergänzung: WB 2.13.9 R24, PHP 8.4

[sternchen8875]

Danke für den Hinweis. 
In der Theorie läuft der Dateiname durch die Funktion page_filename() - heute sanitizeFilename() - und die gibt einen klein geschriebenen Dateinamen zurück. Ich werde mal forschen, was da schief geht. Ich kann mich zwar nicht erinnern, da etwas getan zu haben, aber ein anderer kommt ja nicht in Frage.

Hast aber Recht, das sollte und muß einheitlich sein und wird dann wieder mit Kleinbuchstaben, wie es immer war.

[sternchen8875]

Da hat sich der Dietmar ja was Feines ausgedacht 

Also.... der Dateiname läuft bei der Bearbeitung der Page-Settings dann als seo_title weiter. Das bedeutet, der User darf bestimmen, wie die Datei heißen soll. Damit folgte er anderen Systemen wie Wordpress oder (insbesondere) Joomla, wo das viel verbreitet wurde.
Reich code-technisch ist es so, das in der add.php der eingetragene Page_Title genutzt wird, das die erwähnte Funktion durchläuft, in der die Option CaseSensitive == false ist (per default)

Das läuft dann Analog auch in den Page-Settings, nur ist dort CaseSensitive == true, heißt: der User bestimmt die Schreibweise. Ich erspare mir die Sucherei im Forum, ob es dort mal eine Diskussion gegeben hat, könnte mir aber vorstellen, das das Setzen von CaseSensitive hier in guter Absicht passierte, um ein "Stück Selbstbestimmung" zu überlassen. Eine Rolle spielte dabei mit Sicherheit der Umbau der internen Funktionen und das Zusammenfassen mehrerer Funktionen in einer einzigen, der sanitizeFilename()
Sei es, wie es sei....

Nicht bedacht war dabei wohl die Option, die du verwendet hast, den Dateinamen herauszulöschen. In diesem Fall wird dann wieder auf den Page_Title zugegriffen, der in der Regel mit Großbuchstaben geschrieben wurde.

Was wäre die Lösung?
die Empfehlung geht ganz klar zur Kleinschreibung in Dateinamen. Es gibt auch keine Vor- oder Nachteile in SEO-Dingen. Von daher könnte man auf diese Option, die Schreibweise des Seo_Title selbst zu bestimmen, komplett verzichten, Dateinamen von AccessFiles werden dann immer klein geschrieben - Punkt. Fertig. Aber wir wissen, was da kommt, wenn etwas "weggenommen" wird....

die andere Option wäre es, genau deine Vorgehensweise abzufangen. Ist also kein sog seo_title übergeben, verwende wieder den Page_Title in Kleinbuchstaben

P.S.:
wen es direkt stört....

in der Datei admin/pages/settings2.php in Zeile 238

Original

Code: [Select]

$link = '/'.$parent_section.PreCheck::sanitizeFilename($seo_title,$bCaseSensitiv,$bPageNewstyle);
ändern in

Code: [Select]

$link = '/'.$parent_section.PreCheck::sanitizeFilename($seo_title, false, $bPageNewstyle);

[ruebenwurzel]

Hallo,

danke für die schnelle Reaktion. Leider bringen deine vorgeschlagenen Änderungen in der settings2.php nicht den gewünschten Effekt.

Matthias

[sternchen8875]

wenn es mal schnell gehen soll....    *sorry

da fehlte noch eine Zeile, die genau so korrigiert werden muß. Die obige Korrektur gilt nur für Level 0

die zweite Zeile wäre die Ln 239

Original

Code: [Select]

$link = '/'.$parent_section.PreCheck::sanitizeFilename($seo_title,$bCaseSensitiv,$bPageNewstyle);
geändert auf

Code: [Select]

$link = '/'.$parent_section.PreCheck::sanitizeFilename($seo_title,false,$bPageNewstyle);
der gesamte Block zur Übersicht

Code: [Select]

if ($parent == '0'){
//        $link = '/'.page_filename( $seo_title,$bPageNewstyle);
        $link = '/'.PreCheck::sanitizeFilename($seo_title,false,$bPageNewstyle);
       
        // rename menu titles: index && intro to prevent clashes with intro page feature and WB core file /pages/index.php
        if ($link == '/index' || $link == '/intro'){
            $filename = WB_PATH.PAGES_DIRECTORY.'/'.$link.'_'.$page_id .PAGE_EXTENSION;
            $link .= '_' .$page_id;
        } else {
            $filename = WB_PATH.PAGES_DIRECTORY.'/'.$link.PAGE_EXTENSION;
        }
    } else {
        $parent_section = '';
        $parent_links = \array_reverse(get_parent_links($parent));
        foreach($parent_links as $parent_link) {
      //let the parent as is
//          $parent_section .= page_filename($parent_link, $bPageNewstyle).'/';
        $parent_section .= PreCheck::sanitizeFilename($parent_link,$bCaseSensitiv,$bPageNewstyle).'/';
        } //  end foreach
        if ( $parent_section == '/' ) { $parent_section = ''; }
//        $link = '/'.$parent_section.page_filename($seo_title,$bPageNewstyle);
        $link = '/'.$parent_section.PreCheck::sanitizeFilename($seo_title,false,$bPageNewstyle);
        $filename = WB_PATH.PAGES_DIRECTORY.$link.PAGE_EXTENSION;
    }