Service Button funktioniert nicht mehr / klappt nicht mehr aus

Hi,

ich versuche gerade einige Bugs zu fixen bei jemanden. 

Ich habe nun auch fast alles gelöst, was technisch garnicht ging oder bedingt. Aber der Service Button hat ein massives Problem.

Domain: https://www.ats-racing.de

Er wird irgendwie garnicht geladen (Laut Dev-Tools). In der topbar-navigation.tpl wurde nur ein

eingebaut vom Vorentwickler. Sollte eigentlich alles gehen.

evtl ein .js Problem. das ist aber nicht meine Paradedisziplin, daher etwas schwer das zu finden, falls es das ist.

.less-Seitig wurde garnichts geändert, da der Vorentwickler alles einfach in eine stylesheet.css gepackt hat. Ist etwas blöd aber naja.

Das gleiche Problem gibt es nochmals bei der mobile Ansicht des Anmelden oder Registrierung, wo die Neuanmeldung aufklappen müsste, es aber auch nicht tut. Da habe ich aber noch nicht tiefer gegraben.

 

Evtl. hat schonmal jemand das Problem gehabt oder sieht es sofort. Möchte da nicht noch 20h suchen müssen.

 

Würde mich freuen da einen Tipp zu bekommen, wo ich suchen kann.

 

VG Kay

Es scheint so als hätte der Vorentwickler sehr unsauber gearbeitet. Wenn man sich den Fehler im Browser anzeigt sieht man im Quellcode folgendes  Foot-in-Mouth

document.asyncReady(function () {
alert('hi');
// do your magic here
});

Bist du zufällig auf Shopware 5.5? Das Menü scheint leer zu sein. Evtl. stehen noch die alten Verknüpfungen im Template.

1 „Gefällt mir“

Oh man, von solchen tollen Sachen hab ich schon viel gefunden im Code. Dann suche ich das mal in einer der .tpl ^^

Ja das Update auf 5.5 wurde letzte Woche gemacht.

 

Danke schonmal…

@Shyim schrieb:

Es scheint so als hätte der Vorentwickler sehr unsauber gearbeitet. Wenn man sich den Fehler im Browser anzeigt sieht man im Quellcode folgendes  Foot-in-Mouth

document.asyncReady(function () {
alert(‚hi‘);
// do your magic here
});

 

Das hab ich erstmal raus. Immerhin… 

 

Auch im bare von 5.5 sehen die tpl daten gleich aus. Also ollte das menu ja trotzdem geladen werden. Im backend ist gleft auch zugewiesen. 

Das gleiche Problem hatte hier so ein Schweizer-Italoshop. Guck mal ob dein Theme so aussieht wie hier: shopware/topbar-navigation.tpl at 5.5 · shopware/shopware · GitHub Besonders Zeile 31-34 sind wichtig

1 „Gefällt mir“

@brettvormkopp schrieb:

Das gleiche Problem hatte hier so ein Schweizer-Italoshop. Guck mal ob dein Theme so aussieht wie hier: https://github.com/shopware/shopware/blob/5.5/themes/Frontend/Bare/frontend/index/topbar-navigation.tpl Besonders Zeile 31-34 sind wichtig

Ich danke euch. der Fehler war das dort noch gLeft stand statt left wie es jetzt heisst.

Jetzt klappt es alles wieder. 

habe da dann fast das gleiche Problem auf der Registrierungseite.

Dort klappt im mobilen die „Ich bin Neukunde“ nicht aus um sich anzumelden.

Habe mal alles aus SW 5.5 eingesetzt aber das bringt hier leider nichts.

Der Vorentwikcler hat leider auch in jede einzele tpl aus dem Bare Theme mit ins neue gepackt und in jeder zweiten irgendwas kleines gebastelt. Das ist zum verrückt werden.

Link: https://ats-racing.de/account (in der mobilen Version)

Hallo,

dann fang doch am besten von vorne an, wenn das Theme ansich “nicht zu retten” ist. Schließlich soll es ja updatesicher bleiben und du wirst ja wahrscheinlich den Shop weiter betreuen (wollen). Ansich sollte man nie Sachen aus dem Bare- oder Responsive-Theme kopieren, sondern nur entsprechende Shopware Blöcke anpassen - also solltest du am besten alles kopierte entfernen und nur Anpassungen drin lassen, die auch wirklich nötig sind.

Grüße

Sebastian

@sschreier schrieb:

Hallo,

dann fang doch am besten von vorne an, wenn das Theme ansich „nicht zu retten“ ist. Schließlich soll es ja updatesicher bleiben und du wirst ja wahrscheinlich den Shop weiter betreuen (wollen). Ansich sollte man nie Sachen aus dem Bare- oder Responsive-Theme kopieren, sondern nur entsprechende Shopware Blöcke anpassen.

Grüße

Sebastian

Das nochmal zu bauen habe ich schon vorgeschlagen. Aber kostenlos will ich das auch nicht machen.

Sollten an sich nur kleine Bugfixes werden. 

 

Da sind relativ viele Javascript-Fehler wenn man die Seite auf einem kleinen Viewport öffnet.

[DOM] Found 2 elements with non-unique id #register_personal_password: (More info: https://goo.gl/9p2vKq) ​

Sieht erstmal so aus, als ob da ein Bereich im Theme doppelt geladen wird.

@Moritz Naczenski schrieb:

Da sind relativ viele Javascript-Fehler wenn man die Seite auf einem kleinen Viewport öffnet.

[DOM] Found 2 elements with non-unique id #register_personal_password: (More info: https://goo.gl/9p2vKq) ​

Sieht erstmal so aus, als ob da ein Bereich im Theme doppelt geladen wird.

Okay, geil ^^

dann gehe ich mal auf schnitzeljagt, danke schonmal… 

Hallo,

im Prinzip ist das Theme doch aber nur voller Fehler - hier wäre eigentlich die einzig logische Entscheidung (auch seitens des „Spezialisten“), das Theme neu zu machen. Beispiele für offensichtliche Fehler:

  • im Produkt Listing gibt es „Vergleichen“ und „Merken“ - beim Klick auf „Vergleichen“ kommt die Abdunklung und geht nicht mehr weg, ist der Vergleich aktiv, verschiebt sich der komplette Header. Bei „Merken“ wird überhaupt keine sichtbare Funktion ausgeführt.
  • wählt man im Listing einen Filter aus, wird jede Auswahl doppelt angezeigt.
  • das OffCanvas - Menü in den mobilen Ansichten öffnet sich überhaupt nicht, ist komplett ohne Funktion.
  • die „Sticky“-Funktion des Headers springt merklich, vor allem auf der Produktdetailseite.
  • in den mobilen Ansichten fehlt auf der Produktdetailseite der komplette Bildslider, man hat keine Möglichkeit zur Auswahl eines anderen Bildes.
  • in vielen Bereichen fehlen merkliche Abstände.
  • usw.

In dem Theme ist im Prinzip mehr kaputt als ganz - da kann man als seriöse Entscheidung seitens eines Entwicklers eigentlich nur das Theme neu machen, weils einfach schneller gehen würde als immer wieder zu versuchen, einzelne Bereiche zu reparieren oder zu retten.

Grüße

Sebastian

@sschreier schrieb:

Hallo,

im Prinzip ist das Theme doch aber nur voller Fehler - hier wäre eigentlich die einzig logische Entscheidung (auch seitens des „Spezialisten“), das Theme neu zu machen. Beispiele für offensichtliche Fehler:

  • im Produkt Listing gibt es „Vergleichen“ und „Merken“ - beim Klick auf „Vergleichen“ kommt die Abdunklung und geht nicht mehr weg, ist der Vergleich aktiv, verschiebt sich der komplette Header. Bei „Merken“ wird überhaupt keine sichtbare Funktion ausgeführt.
  • wählt man im Listing einen Filter aus, wird jede Auswahl doppelt angezeigt.
  • das OffCanvas - Menü in den mobilen Ansichten öffnet sich überhaupt nicht, ist komplett ohne Funktion.
  • die „Sticky“-Funktion des Headers springt merklich, vor allem auf der Produktdetailseite.
  • in den mobilen Ansichten fehlt auf der Produktdetailseite der komplette Bildslider, man hat keine Möglichkeit zur Auswahl eines anderen Bildes.
  • in vielen Bereichen fehlen merkliche Abstände.

In dem Theme ist im Prinzip mehr kaputt als ganz - da kann man als seriöse Entscheidung seitens eines Entwicklers eigentlich nur das Theme neu machen, weils einfach schneller gehen würde als immer wieder zu versuchen, einzelne Bereiche zu reparieren oder zu retten.

Grüße

Sebastian

ach du sch****

das nimmt ja kein Ende. Das behebe ich nicht mehr, das stimmt.  

neues theme steht nicht zur debatte… 

soll nur den fehler fixen und den rest lassen 

 

jemand eine idee was die js fehler bringt?

@Moritz Naczenski schrieb:

Da sind relativ viele Javascript-Fehler wenn man die Seite auf einem kleinen Viewport öffnet.

[DOM] Found 2 elements with non-unique id #register_personal_password: (More info: https://goo.gl/9p2vKq) ​

Sieht erstmal so aus, als ob da ein Bereich im Theme doppelt geladen wird.

gibt es die Möglichkeit, zu sehen welche .tpl diese Parts läd? 

konnte es auf die index.tpl eingrenzen

{extends file="parent:frontend/index/index.tpl"} {block name="frontend_index_start"}{/block} {block name="frontend_index_doctype"} {/block} {block name='frontend_index_html'}

{/block} {block name='frontend_index_header'} {include file='frontend/index/header.tpl'} {/block}

{block name='frontend_index_after_body'}{/block} {block name="frontend_index_page_wrap"}

{* Message if javascript is disabled *} {block name="frontend_index_no_script_message"} {/block} {block name='frontend_index_before_page'}{/block} {* Shop header *} {block name='frontend_index_navigation'}
{* Logo container *} {block name='frontend_index_logo_container'} {include file="frontend/index/logo-container.tpl"} {/block}
{* Shop navigation *} {block name='frontend_index_shop_navigation'} {include file="frontend/index/shop-navigation.tpl"} {/block}

{* Include the top bar navigation *} {block name='frontend_index_top_bar_container'} {include file="frontend/index/topbar-navigation.tpl"} {/block}
{* Logo container *} {block name='frontend_index_logo_container'} {include file="frontend/index/logo-container.tpl"} {/block}
{* Shop navigation *} {block name='frontend_index_shop_navigation'} {include file="frontend/index/shop-navigation.tpl"} {/block}
{block name='frontend_index_header_navigation'} {* Maincategories navigation top *} {block name='frontend_index_navigation_categories_top'}
{block name="frontend_index_navigation_categories_top_include"} {include file='frontend/index/main-navigation.tpl'} {/block}
{/block} {/block}
{block name='frontend_index_container_ajax_cart'}

{/block}
{/block}
{block name='frontend_index_emotion_loading_overlay'} {if $hasEmotion}

{/if} {/block}
{block name='frontend_index_content_main'}{* Breadcrumb *} {block name='frontend_index_breadcrumb'} {if count($sBreadcrumb)}
{block name='frontend_index_breadcrumb_inner'} {include file='frontend/index/breadcrumb.tpl'} {/block}

{/if} {/block} {* Content top container *} {block name="frontend_index_content_top"}{/block}
{* Sidebar left *} {block name='frontend_index_content_left'} {include file='frontend/index/sidebar.tpl'} {/block} {* Main content *} {block name='frontend_index_content_wrapper'}
{block name='frontend_index_content'}{/block}
{/block} {* Sidebar right *} {block name='frontend_index_content_right'}{/block} {* Last seen products *} {block name='frontend_index_left_last_articles'} {if $sLastArticlesShow && !$isEmotionLandingPage} {* Last seen products *}
{s namespace="frontend/plugins/index/viewlast" name='WidgetsRecentlyViewedHeadline'}{/s}

{/if} {/block}
{/block} {* Sidebar left *} {block name='frontend_index_content_left'} {include file='frontend/index/sidebar.tpl'} {/block} {* Main content *} {block name='frontend_index_content_wrapper'}
{block name='frontend_index_content'}{/block}
{/block}
{* Footer *} {block name="frontend_index_footer"}
{block name="frontend_index_footer_container"} {include file='frontend/index/footer.tpl'} {/block}
{/block} {block name='frontend_index_body_inline'}{/block}
{/block} {* If required add the cookiePermission hint *} {block name='frontend_index_cookie_permission'} {if {config name="show_cookie_note"}} {include file="frontend/_includes/cookie_permission_note.tpl"} {/if} {/block} {block name="frontend_index_ajax_seo_optimized"} {* @deprecated SEO support for AJAX routes is deprecated in 5.4. You can disable the check for AJAX routes in Shopware 5.4 by overriding this block and setting the variable to false. This block will be removed in Shopware 5.5 and all affected AJAX routes below will have SEO support disabled. *} {$ajaxSeoSupport = true} {/block} {block name="frontend_index_header_javascript"} {$controllerData = ['vat_check_enabled' => {config name='vatcheckendabled'}, 'vat_check_required' => {config name='vatcheckrequired'}, 'register' => {url controller="register"}, 'checkout' => {url controller="checkout"}, 'ajax_search' => {url controller="ajax_search" _seo=false}, 'ajax_cart' => {url controller='checkout' action='ajaxCart' _seo=$ajaxSeoSupport}, 'ajax_validate' => {url controller="register" _seo=$ajaxSeoSupport}, 'ajax_add_article' => {url controller="checkout" action="addArticle" _seo=$ajaxSeoSupport}, 'ajax_listing' => {url module="widgets" controller="Listing" action="ajaxListing" _seo=$ajaxSeoSupport}, 'ajax_cart_refresh' => {url controller="checkout" action="ajaxAmount" _seo=$ajaxSeoSupport}, 'ajax_address_selection' => {url controller="address" action="ajaxSelection" fullPath _seo=$ajaxSeoSupport}, 'ajax_address_editor' => {url controller="address" action="ajaxEditor" fullPath _seo=$ajaxSeoSupport}]} {$themeConfig = ['offcanvasOverlayPage' => $theme.offcanvasOverlayPage]} {$lastSeenProductsKeys = []} {foreach $sLastArticlesConfig as $key => $value} {$lastSeenProductsKeys[$key] = $value} {/foreach} {$lastSeenProductsConfig = ['baseUrl' => $Shop->getBaseUrl(), 'shopId' => $Shop->getId(), 'noPicture' => {link file="frontend/_public/src/img/no-picture.jpg"}, 'productLimit' => {"{config name=lastarticlestoshow}"|floor}, 'currentArticle' => ""]} {if $sArticle} {$lastSeenProductsConfig.currentArticle = $sLastArticlesConfig} {$lastSeenProductsConfig.currentArticle.articleId = $sArticle.articleID} {$lastSeenProductsConfig.currentArticle.linkDetailsRewritten = $sArticle.linkDetailsRewrited} {$lastSeenProductsConfig.currentArticle.articleName = $sArticle.articleName} {if $sArticle.additionaltext} {$lastSeenProductsConfig.currentArticle.articleName = $lastSeenProductsConfig.currentArticle.articleName|cat:' ':$sArticle.additionaltext} {/if} {$lastSeenProductsConfig.currentArticle.imageTitle = $sArticle.image.description} {$lastSeenProductsConfig.currentArticle.images = []} {foreach $sArticle.image.thumbnails as $key => $image} {$lastSeenProductsConfig.currentArticle.images[$key] = ['source' => $image.source, 'retinaSource' => $image.retinaSource, 'sourceSet' => $image.sourceSet]} {/foreach} {/if} {$csrfConfig = ['generateUrl' => {url controller="csrftoken" fullPath=false}, 'basePath' => $Shop->getBasePath(), 'shopId' => $Shop->getId()]} {* let the user modify the data here *} {block name="frontend_index_header_javascript_data"}{/block} {include file="frontend/index/datepicker-config.tpl"} {if $theme.additionalJsLibraries} {$theme.additionalJsLibraries} {/if} {/block} {block name="frontend_index_header_javascript_jquery"} {* Add the partner statistics widget, if configured *} {if !{config name=disableShopwareStatistics} } {include file='widgets/index/statistic_include.tpl'} {/if} {/block} {* Include jQuery and all other javascript files at the bottom of the page *} {block name="frontend_index_header_javascript_jquery_lib"} {compileJavascript timestamp={themeTimestamp} output="javascriptFiles"} {foreach $javascriptFiles as $file} {/foreach} {/block} {block name="frontend_index_javascript_async_ready"} {include file="frontend/index/script-async-ready.tpl"} {/block}

 

Hallo,

du wirst doch dafür entlohnt, dann solltest du dich auch damit auseinandersetzen, wenn du das Theme so (defekt) belassen möchtest. Ebenso seid ihr doch auch “Spezialisten”.

Grüße

Sebastian

Also im aktuellen Firefox funktioniert das Service Dropdown. Aber darunter tun sich weitere Abgründe auf: Keine Datenschutz- bzw. nur Fülltexte zu habe ist zur Zeit ein bisschen fahrlässig Foot-in-Mouth Dazu hüpft die Datenschutzseite beim Scrollen ziemlich komisch herum.

@puhas schrieb:

Also im aktuellen Firefox funktioniert das Service Dropdown. Aber darunter tun sich weitere Abgründe auf: Keine Datenschutz- bzw. nur Fülltexte zu habe ist zur Zeit ein bisschen fahrlässig Foot-in-Mouth Dazu hüpft die Datenschutzseite beim Scrollen ziemlich komisch herum.

Das Service Kram ist behoben worden.

Ja es sind viele Fehler drin. Mit viel Überzeugung wird es jetzt neu aufgesetzt.