[BUG] content_link_list - Link nicht löschbar

[rethus]

Ich finde keine Möglichkeit, einen über das Modul content_link_list eingegebenen Link zu löschen.
Gehe ich auf den Link, kann ich zwar in dem Verzeichnis "kein" wählen, und den "Link-Title" löschen, dennoch bleibt der Link zum upload-Verzeichnis (in kompletter schreibweise http://domain.tld/uploads/) dort stehen.

Öffne ich diesen Link erneut, ist auch noch die Selektion von "Kein" im Dateifenster zu sehen.
Hier hat das Modul ein Bug, der scheinbar einige Leichen in der DB zurück lässt. Gut wäre das Modul um einen Löschbutton zu ergänzen, der den Link entfernt.

[rethus]

Hier ein einfacher Lösungsansatz, ohne Bereinigungs-Funktion in der DB:

In der Datei:class.content.type.linkeditor.php dort in der Funktion generateViewCode() folgendes:

Code: Alles auswählen

        // generate the needed attributes
        $href = $this->_generateHref();
        if (empty($href)) {
            return '';
        }
 

ersetzen gegen:

Code: Alles auswählen

 
        // generate the needed attributes
        $href = $this->_generateHref();
        if (empty($href) || $href == $href == $this->_cfgClient[$this->_client]["upl"]["htmlpath"]) {
            return '';
        }
 

Sauberer wäre es, an dieser Stelle die gefundene Leiche aus der DB auch direkt zu entfernen, nur fehlt mir hier leider gerade die Zeit dazu. Daher nur dieses Feedback mit Bitte, dass in dem Modul (zum nächsten Release) zu optimieren.

[rethus]

Hier noch ein Verbesserungsvorschlag für das Modul (also in Style→Module→content_link_list):

vor $tpl->display('get.tpl'); noch folgendes einfügen, damit das Template gar keine HTML-Container erstellt, wenn keine Inhalte vorhanden sind:

Code: Alles auswählen

if(cRegistry::isBackendEditMode() || count(array_filter($val))>0)

[Oldperl]

Code: Alles auswählen

if (empty($href) || $href == $href == $this->_cfgClient[$this->_client]["upl"]["htmlpath"]) {

Ist das so richtig mit dem 2x $href ==?

Gruß aus Franken

Ortwin

[rethus]

Ja, für den Nachdruck

Ne, hast natürlich recht, war ein Vertipper, hatte zuvor die cfgClient und $client über globals in die Funktion geholt. Da aber die Klasse diese als Variable besitzt, hatte ich die ersetzt und wohl versehentlich ein href zu viel reingehauen.

So is richtig:

Code: Alles auswählen

if (empty($href) || $href == $this->_cfgClient[$this->_client]["upl"]["htmlpath"]) {

PS: Schade im Forum nur zutierte CODE-Segmente gehighlightet werden. Könnte 4fb doch mal dran schrauben, dass als CODE-Deklarierte Blöcke auch syntax-highlighting aktiviert wird.

[Oldperl]

...dass als CODE-Deklarierte Blöcke auch syntax-highlighting aktiviert wird.

Das geht schon, du must nur im CodeTag ein =php ergänzen. Wobei der Parser sicherlich etwas besser sein könnte. Guckst du hier

Code: Alles auswählen

function HighLightMe($inFarbe) {

    return ($inFarbe) ? $sAllesSoBunt : FALSE;
}

class machEtBunt extends FarbeImSpiel {

       public function __construct() {
           echo "Schoen bunt hier";
      }
}

ODER

Code: Alles auswählen

if (empty($href) || $href == $this->_cfgClient[$this->_client]["upl"]["htmlpath"]) { 

Gruß aus Franken

Ortwin

[frederic.schneider_4fb]

Hallo Jungs,

danke für die Anregungen. Ich wollte mir gerade Euer Problem ansehen mit der Bereitschaft, die vorgenannten Code-Änderungen ggfls. in den Core zu übernehmen. Dabei wollte ich jedoch erst einmal das Problem nachstellen - und habe festgestellt, dass ich das Problem nicht habe. Vielleicht habe ich Euch aber auch nur missverstanden:
Ich habe in einer Linklist einen internen Link angegeben (Auswahl eines Verzeichnis). Im Anschluss habe ich den Link bearbeitet und habe statt den ausgewählten internen Ordner "Keine" ausgewählt sowie den Linktitel entfernt. Rufe ich danach die Bearbeitung auf, ist es m. E. alles wie es soll: Der Link ist weg. (Bei externen Links verhält es sich ebenfalls wie es soll.) Bitte korrigiert mich, wenn ich mich auf einem falschen Pfad bewege .

[rethus]

Bei mir ist es halt so, dass der Link scheinbar nicht wirklich gelöscht wird, sonder nur der Dateiname. Der Pfad zum Upload-Verzeichnis bleibt erhalten, und daher wird auch was im Frontend ausgegeben. Mit der obigen Anpassung läuft es aber sauber (wenn auch die Leiche in der DB immer noch vorhanden ist).

[frederic.schneider_4fb]

Wie erwähnt: Ich habe Sympathie für Eure Anregung, habe aber auch ohne die aufgezeigten Lösungswege gar nicht das Problem. Habe ich in meinem Reproduzierungsvorgang (siehe Schilderung) irgend etwas "falsch" gemacht?