URL Alias bei Contentseiten

modellbau

Hi zusammen,

ich habe vermutlich einen Fehler entdeckt. Wenn man eine Contentseite anlegt und eine url_alias angibt und abspeichert funktioniert alles wie gewünscht. Ändert man nun nachträglich die Contentseite, wird die url_alias nicht mehr genommen stattdessen der title. Den Fehler konnte ich beheben indem ich in der inc/commerce_seo.inc.php folgenden Code geändert habe:

PHP

if($contentList['content_url_alias'] !='')

ersetzt durch:

PHP

if($content['content_url_alias'] !='')

Kann der Fehler und die Lösung noch von einem der Admins bestätigt werden?

Viele samstägliche Grüße!
Benny

modellbau

Mist: bei jedem zweiten mal Abspeichern hängt er mir die LanguageId noch an den url_alias ran... warum denn das?

modellbau

ok, jetzt hab ich auch diesen Bug. Hier nochmals den kompletten Code:

PHP

case 'content':
				$content_query = "SELECT
										cm.content_group,
										cm.languages_id,
										cm.content_title,
										cm.content_url_alias,
										l.code
									FROM
										" . TABLE_CONTENT_MANAGER . " cm,
										". TABLE_LANGUAGES ." l
									WHERE
										cm.content_group = '".$id."'
									AND
										cm.languages_id = l.languages_id";


		   	 	$content_query = xtDBquery($content_query);




				while($content = xtc_db_fetch_array($content_query, false)) {

					if($content['content_url_alias'] !='') 
						$content_url = $content['content_url_alias'];
					else
						$content_url = $content['content_title'];


					if($useLanguageUrl)
						$contentLink = $content['code'].'/'.cseo_get_url_friendly_text($content_url);
					else
						$contentLink = cseo_get_url_friendly_text($content_url);


                $old_contentlink = xtc_db_fetch_array(xtDBquery(" SELECT url_text
																			FROM commerce_seo_url
																			WHERE content_group='".$id."'
																			AND language_id='".$content['languages_id']."'"));

					if($contentLink != $old_contentlink['url_text']) {
					$contentLink = $this->validateDBKeyLink ($contentLink,'');
					$content_seo_query = xtDBquery("UPDATE commerce_seo_url SET url_md5 = '".md5($contentLink)."', url_text = '".$contentLink."' WHERE content_group = '".$id."' AND language_id='".$content['languages_id']."'");
					}
				}

Alles anzeigen

admin

Bin dran. Kann momentan den Fehler nur teilweise reproduzieren.

pqmagic

Kann den Bug bestätigen. Teilweise wird sogar bei aktivierter alias URL eine 1 am Ende der URL. Haben schon versucht die alias URL abzuschalten, brachte aber das gleiche Ergebnis.

admin

Ist im FP2 behoben. Der Bug, bei Mehrsprachigkeit, dass da teilweise beim Speichern ein Fehler kommt, sollte dann auch raus sein.

modellbau

Vielen Dank!

admin

Jou, habe das gestern noch getestet in allen Kombis und scheint jetzt komplett fehlerfrei zu sein .-)

pqmagic

Hi Admin,

kannst du posten was da genau geändert worden ist? Das Einspielen der FP´s birgt beim laufenden Shop immer das gewisse Risko

admin

in der inc/commerce_seo.inc.php
Zeile ab: 674

Code

while($content = xtc_db_fetch_array($content_query, false)) {
                    if($contentList['content_url_alias'] !='')
                        $content_url = $content['content_url_alias'];
                    else
                        $content_url = $content['content_title'];


                    if($useLanguageUrl)
                        $contentLink = $content['code'].'/'.cseo_get_url_friendly_text($content_url);
                    else
                        $contentLink = cseo_get_url_friendly_text($content_url);


                    $contentLink = $this->validateDBKeyLink ($contentLink,'');


                    $content_seo_query = xtDBquery("UPDATE commerce_seo_url SET url_md5 = '".md5($contentLink)."', url_text = '".$contentLink."' WHERE content_group = '".$id."' AND language_id='".$content['languages_id']."'");
                }

Alles anzeigen

neu:

Code

while($content = xtc_db_fetch_array($content_query, false)) {
                    if($content['content_url_alias'] !='')
                        $content_url = $content['content_url_alias'];
                    else
                        $content_url = $content['content_title'];


                    if($useLanguageUrl)
                        $contentLink = $content['code'].'/'.cseo_get_url_friendly_text($content_url);
                    else
                        $contentLink = cseo_get_url_friendly_text($content_url);


                    $contentLink = $this->validateDBKeyLink ($contentLink,'');


                    $content_seo_query = xtDBquery("UPDATE commerce_seo_url SET url_md5 = '".md5($contentLink)."', url_text = '".$contentLink."' WHERE content_group = '".$content['content_group']."' AND language_id='".$content['languages_id']."'");
                }

Alles anzeigen

Benutzer online in diesem Thema