Verbesserungsvorschlag: Content-Manager Links in TOPNAVI

[matze]

Hallo,

beim Überarbeiten unseres Shop-Templates habe ich als TopNavi ein mehrstufiges DropDownMenü eingefügt. In dem Menü werden einige Einträge aus dem Content-Manager als Link in der index.html eingefügt. Um das ganze auch mit mehreren Sprachen vernünftig zu machen, müssen natürlich die Bezeichnungen auch aus der Datenbanktabelle content_manager mit der jeweils aktuellen Sprache eingesetzt werden.

Dazu habe ich die Funktion xtc_get_content_title($coID) in der Datei /inc/xtc_get_content_title.inc.php erstellt.

Hier ein Beispiel: Im Content-Manager gibt es in den Sprachen Deutsch, Englisch und Spanisch jeweils einen Eintrag mit der Sprachgruppe 20. Im Feld Titel steht bei Deutsch "Workshop & Schulungen", bei Englisch "Workshops & courses" und bei Spanisch "Workshops & Cursos".

Der Link wird dann folgendermaßen in der index.html erzeugt:

Code: PHP  [Auswählen]

<a href="{php}echo xtc_href_link(FILENAME_CONTENT,'coID=20','NONSSL');{/php}>{php}echo xtc_get_content_title('20');{/php}</a>

Damit wird auch bei Wechsel der Shop-Sprache im Menü immer der Linktext in der gewählten Sprache angezeigt.

Würde mich freuen wenn diese Erweiterung in den normalen Release aufgenommen werden könnte.

Matthias



Linkback: https://www.modified-shop.org/forum/index.php?topic=9558.0

[Tomcraft]

Eine schöne Idee!

Das nehme ich mal auf die Roadmap für spätere Versionen mit auf.

Grüße

Torsten

[matze]

Hallo,

hier der neue Include /inc/xtc_get_content_title.inc.php

Code: PHP  [Auswählen]

<?php
/* --------------------------------------------------------------
   configuration_get_content_title.inc.php MT;21.11.2010
   Tenoch
   http://www.tenoch.de
   Copyright (c) 2010 Tenoch
   Released under the GNU General Public License
   --------------------------------------------------------------
*/
?><?php

        /*
                -> function to get content title
        */

        function xtc_get_content_title($coID) {

                                        $query = xtc_db_query("
                                                                                        SELECT
                                                                                                content_title
                                                                                        FROM
                                                                                                content_manager
                                                                                        WHERE
                                                                                                content_group = '" . $coID . "'
                                                                                                AND languages_id = '" . $_SESSION['languages_id'] . "' LIMIT 1
                                                                                        ");
                                                $row = xtc_db_fetch_array($query);
                                                $values = $row['content_title'];
                return $values;
        }
?>
 

und das Einbinden in /include/header.php:

Code: PHP  [Auswählen]

  // include needed functions
  require_once('inc/xtc_output_warning.inc.php');
  require_once('inc/xtc_image.inc.php');
  require_once('inc/xtc_parse_input_field_data.inc.php');
  require_once('inc/xtc_draw_separator.inc.php');

//  require_once('inc/xtc_draw_form.inc.php');
//  require_once('inc/xtc_draw_pull_down_menu.inc.php');
//MT;21.11.2010
  require_once('inc/xtc_get_content_title.inc.php');
 

[ Für Gäste sind keine Dateianhänge sichtbar ]

[matze]

Nochmals Hallo,

lege auch noch die Source dazu.

Gruß Matze

[web4design]

Hi.

Bitte sichere "$_SESSION['languages_id']" wenigstens mit nem "(int)" ab.

Gruß

[Tomcraft]

Hübsch geworden!

Nachtrag: Er hat ja noch 25 Minuten Zeit seinen Beitrag zu editieren.

Grüße

Torsten

[GTB]

Hallo Matze,

Schöne Lösung.

Habe mir gerade deinen Shop angeschaut. Die Schrift der Breadcrumb ist zu groß ...

Gruss Gerhard

[matze]

Hallo Gerhard,

danke für den Tipp, werde mal das CSS anpassen. Ist aus einem modifizierten xtc5 Template.

Gruß Matze

[matze]

Hallo  speed4project ,

verstehe deinen Vorschlag nicht ganz, auch wenn $_SESSION['languages_id'] leer ist, ergibt sich im SQL Query ein leeres ''. Das erzeugt doch keinen SQL Fehler, liefert dann lediglich ein leeres Resultat zurück.

Gruß Matthias

[web4design]

Hi.

Die Passage:

Code: PHP  [Auswählen]

'" . $_SESSION['languages_id'] . "'

sollte zumindest so aussehen:

Code: PHP  [Auswählen]

'" . (int)$_SESSION['languages_id'] . "'

xtc_db_query setzt einfach die DB Abfrage ab, da wird nichts mehr maskiert, überprüft etc. Hat den Nachteil, dass eine SQL-Injection denkbar ist. Das trifft man in sehr vielen selbst programmierten und auch veröffentlichten Erweiterungen, dass genau darauf nicht geachtet wird.

Ich könnte mich auch irren

Gruß

[matze]

Hallo speed4project,

das klingt logisch. werde den (int) integrieren. Die Routine wurde aus dem Core Teil übernommen und von mir modifiziert, Wenn dies eine Sicherheitslücke ist, dann gibts bestimmt noch mehr in modified eCommerce Shopsoftware.

Gruß Matze

[GTB]

das machen wir den ganzen Tag. Nach Fehlern im System suchen ...  

[Tomcraft]

Den Rest vom Tag schauen wir dann noch im Forum vorbei.

Grüße

Torsten

[matze]

Hallo Torsten,

und Nachts?

[Tomcraft]

Sollte man eigentlich zum Schlafen verwenden, lässt sich nur nicht immer so umsetzen.

Grüße

Torsten