Project

General

Profile

GX-Bug #68368

Leere URL in Paketdienst führt zu Fehler in Hermes HSI | Missing URL in parcel service leads to errors in Hermes HSI

Added by Stefanie Bruchmann over 1 year ago. Updated 11 months ago.

Status:
Erledigt
Priority:
Normal
Assignee:
-
Category:
Core
Target version:
% Done:

0%

Estimated time:
Steps to reproduce:
Betroffene Versionen:
4.3.3.1, 4.4.0.0, 4.4.0.0 beta1
Release Notes Langtext:

Hermes HSI: Konfiguration kann jetzt auch aufgerufen werden, wenn bei einem Paketdienst kein URL angebeben ist


Description

Leere URL in Paketdienst führt zu Fehler in Hermes HSI

Wenn unter Shop Einstellungen -> Paketdienste ein Paketdienst ohne URL angelegt wird, führt dies bei Aufruf des Moduls Hermes HSI zu einer Fehlermeldung:

Uncaught InvalidArgumentException: NonEmptyStringType: Invalid argument value given (expected non-empty string got string): in src/GXEngine/Shared/Types/NonEmptyStringType.inc.php:38
Stack trace:
#0 src/GXMainComponents/Services/Core/ParcelService/Entities/ParcelServiceDescription.inc.php(63): NonEmptyStringType->__construct()
#1 src/GXMainComponents/Services/Core/ParcelService/Repositories/ParcelServiceMapper.inc.php(184): ParcelServiceDescription::create()
#2 src/GXMainComponents/Services/Core/ParcelService/Repositories/ParcelServiceMapper.inc.php(160): ParcelServiceMapper->createParcelService()
#3 src/GXMainComponents/Services/Core/ParcelService/Repositories/ParcelServiceMapper.inc.php(51): ParcelServiceMapper->createParcelServiceCollection()

Testen:

  • Shop Einstellungen -> Paketdienste aufrufen
  • Leeren URL bei einem Paketdienst eintragen
  • Hermes HSI unter Module -> Modul-Center installieren
  • HSI-Konfiguration aufrufen
  • erwartetes Verhalten: hier sollte nun die Modul-Konfiguration (siehe Screenshot) angezeigt werden
  • tatsächliches Verhalten: Ausgabe der oben genannten Fehlermeldung

Der Punkt Shop Einstellungen -> Paketdienste erlaubt das Anlegen von Paketdiensten ohne Angabe einer URL und das ist auch korrekt so, da es hierfür Anwendungsfälle in der Praxis gibt. Der in der Fehlermeldung genannte Service verlangt hier aber eine Eingabe und reagiert mit einer Fehlermeldung, wenn das Feld leer ist. Der Service muss hier daher auch leere Angaben akzeptieren.

===================================================================================================

If a parcel service without a URL is created under Shop Settings -> Parcel Services, this leads to an error message when calling the Hermes HSI module:

Uncaught InvalidArgumentException: NonEmptyStringType: Invalid argument value given (expected non-empty string got string): in src/GXEngine/Shared/Types/NonEmptyStringType.inc.php:38
Stack trace:
#0 src/GXMainComponents/Services/Core/ParcelService/Entities/ParcelServiceDescription.inc.php(63): NonEmptyStringType->__construct()
#1 src/GXMainComponents/Services/Core/ParcelService/Repositories/ParcelServiceMapper.inc.php(184): ParcelServiceDescription::create()
#2 src/GXMainComponents/Services/Core/ParcelService/Repositories/ParcelServiceMapper.inc.php(160): ParcelServiceMapper->createParcelService()
#3 src/GXMainComponents/Services/Core/ParcelService/Repositories/ParcelServiceMapper.inc.php(51): ParcelServiceMapper->createParcelServiceCollection()

Testing:

  • Go to shop settings -> parcel services
  • Enter an empty URL for a parcel service
  • Install Hermes HSI under Modules -> Module Center
  • Call up HSI configuration
  • Expected behavior: the module configuration (see screenshot) should now be displayed here
  • Actual behavior: Output of the above error message

The point Shop Settings -> Parcel Services allows the creation of parcel services without specifying a URL and that is correct, as there are use cases for this in practice. The service named in the error message requires an entry here and reacts with an error message if the field is empty. The service must therefore also accept missing information here.


Files

2021-03-03_004.png (10.9 KB) 2021-03-03_004.png Stefanie Bruchmann, 03/03/2021 01:50 PM

History

#1

Updated by Stefanie Bruchmann over 1 year ago

Marco dazu:

src/GXMainComponents/Services/Core/ParcelService/Entities/ParcelServiceDescription.inc.php ist das Problem, da ist der URL als NonEmptyString behandelt.

#2

Updated by Stefanie Bruchmann over 1 year ago

  • Subject changed from Fehlende URL in Paketdienst führt zu Fehler in Hermes HSI | Missing URL in parcel service leads to errors in Hermes HSI to Leere URL in Paketdienst führt zu Fehler in Hermes HSI | Missing URL in parcel service leads to errors in Hermes HSI
  • Description updated (diff)
#3

Updated by Henning Bödecker over 1 year ago

  • Assignee set to Henning Bödecker
#4

Updated by Henning Bödecker over 1 year ago

  • Status changed from Gemeldet to Anerkannt
  • Assignee deleted (Henning Bödecker)
  • Betroffene Versionen 4.4.0.0 beta1 added
#5

Updated by Marco Bruchmann over 1 year ago

  • Status changed from Anerkannt to In Bearbeitung
  • Assignee set to Marco Bruchmann
  • Betroffene Versionen 4.4.0.0 added
#6

Updated by Marco Bruchmann over 1 year ago

  • Status changed from In Bearbeitung to 30
  • Assignee changed from Marco Bruchmann to Core-Entwickler
  • Target version changed from 133 to 4.5.1.0 beta1
  • Release Notes Langtext updated (diff)
#8

Updated by Daniel Wu about 1 year ago

  • Tags set to 4.4.0.1
#9

Updated by Alexandre Ataides about 1 year ago

  • Assignee changed from Core-Entwickler to Alexandre Ataides
#10

Updated by Alexandre Ataides about 1 year ago

  • Assignee changed from Alexandre Ataides to Core-Entwickler
#11

Updated by Alexandre Ataides about 1 year ago

  • Status changed from 30 to Testing
#12

Updated by Ahmed Hamouda about 1 year ago

  • Status changed from Testing to Erledigt
#13

Updated by Ahmed Hamouda about 1 year ago

  • Target version changed from 4.5.1.0 beta1 to 4.4.0.1
#14

Updated by Daniel Wu 12 months ago

  • Assignee deleted (Core-Entwickler)
#15

Updated by Moritz Bunjes 11 months ago

  • Tags deleted (4.4.0.1)

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 5 MB)