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

Advanced Logging (1/4) - Update ReadOrdersService.php #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

XXL-Webdesign
Copy link

Removed / commented line 89 (90) because of adding AdvandedLog in /Library/API/ApiClient.php

Removed / commented line 89 (90) because of adding AdvandedLog in /Library/API/ApiClient.php
@XXL-Webdesign XXL-Webdesign changed the title Update ReadOrdersService.php Update ReadOrdersService.php - Advanced Logging Part 1/2 Dec 7, 2020
@XXL-Webdesign XXL-Webdesign changed the title Update ReadOrdersService.php - Advanced Logging Part 1/2 Advanced Logging (1/2) - Update ReadOrdersService.php Dec 7, 2020
@XXL-Webdesign XXL-Webdesign changed the title Advanced Logging (1/2) - Update ReadOrdersService.php Advanced Logging (1/4) - Update ReadOrdersService.php Dec 8, 2020
@fatchip-Stefan
Copy link
Collaborator

Hallo, vielen Dank für die PRs,

wir haben alle 4 PRs in einen neuen Branch 76280_advanced_logging eingepflegt und noch einige Anpassungen vorgenommen:
siehe folgende Vergleiche:
(Modul:)
https://github.com/FATCHIP-GmbH/plugin-shopware5-shopware2afterbuy/compare/76280_advanced_logging
(Submodul)
https://github.com/FATCHIP-GmbH/library-afterbuy/compare/76280_advanced_logging

Das Pre-Release Modul 1.0.4 enthält zum Testen Eure Änderungen:
https://github.com/FATCHIP-GmbH/plugin-shopware5-shopware2afterbuy/releases/tag/v1.0.4

Noch einige Anmerkungen:
Das Repo mit dem API Client ist als git submodul angelegt und liegt in einem getrennten repo, s.o.
Ihr könnt beide Repos gleichzeitig mit --recurse submodules auschecken, dann ist der Library/API Ordner auch gefüllt

git clone --recurse-submodules [email protected]:FATCHIP-GmbH/plugin-shopware5-computop.git FatchipCTPayment

Danach muss im Library/API Ordner auch noch mit git checkout einen Branch ausgecheckt werden, weil dieser nach dem clone auf HEAD steht.

Zu den Modul Änderungen:

  • Ich habe die neue Backend Option an den API Client angebunden, so das der Log Level jetzt über die Config im Backend gesetzt werden kann
  • Log Level 1 habe ich so interpretiert, das nur auf diesem Log Level 1 die von Euch komplett auskommentierte Meldung "No data received" geloggt werden soll. Siehe
    if(!$data && $this->config['advLogLevel'] === "1") {
  • Die Beschreibung der Log Level habe ich im API client an die property $advLoglevel übernommen und die anderen Kommentare entfernt, da diese denke ich nicht mehr gebraucht werden

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