Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AB auction view - direct link to product and show articlenumber - Update WriteOrdersService.php #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

XXL-Webdesign
Copy link

ab_directlink
Show article number instead of EAN in aferbuy auction view and add direct link - Update WriteOrdersService.php

Changelog 2020-12-08
modified afterbuy order request
-> shows article number in afterbuy auction overview instead of EAN
-> direct link to the shop product
-> similar behavior as with the TBE-Net plugin before

…rect link - Update WriteOrdersService.php

Changelog 2020-12-08
modified afterbuy order request
-> shows article number in afterbuy auction overview instead of EAN
-> direct link to the shop product
-> similar behavior as with the TBE-Net plugin before
Bugfix:
If Artikelnr_XX is not numeric, you will get an error from afterbuy response.
So now it removes all from articlenumber, which is not numeric.
@doitauto
Copy link

Hi,

soweit alles gut aber Link zum Artikel ist nicht Korrekt, es wird folgendes Link generiert:
https://search/?sSearch=37415795

@XXL-Webdesign
Copy link
Author

Hi,

soweit alles gut aber Link zum Artikel ist nicht Korrekt, es wird folgendes Link generiert:
https://search/?sSearch=37415795

Steht auch in den Kommentaren in der ToDo Liste, dass das noch nicht 100% sauber gelöst ist.

In Zeile 151 "$_SERVER['SERVER_NAME']" durch "$_SERVER['HTTP_HOST']" oder die direkte Webseiten-URL ersetzen sollte Abhilfe schaffen.
Mag Webhoster-basiert Unterschiede geben. Bei uns funktioniert das mit dem "SERVER_NAME".
Langfristig besser ist die Variable aus der Shopware-Config direkt auszulesen.
Hab das aber in meinen Tests so nicht hin bekommen. Entweder hab ich die falschen Befehle genutzt oder es müssen noch includes bzw. uses mit integriert werden um die entsprechenden Variablen auszulesen.
Spätestens mit der direkten Adresse funktioniert es auf jeden Fall. (Das "https://" wird ja bereits mit übertragen.)
Nachteil ist daran nur, dass das Script dann nur noch für die eigene Webseite gültig ist und für jede Webseite entsprechend im Quellcode angepasst werden muss. Aber solange es eh nicht fix im offiziellen Release integriert ist, muss der Zusatz-Code ja eh manuell hinzugefügt werden.

Die Lösung mit dem auslesen aus der Shopware-Config wäre auch in sofern besser, weil man dann auch Shops in Unterverzeichnissen mit berücksichtigen kann, sofern der Shop nicht per Root-URL aufgerufen wird.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants