Acked-by: Henning Leutz <leutz@pcsg.de>

The `.gitlab-ci.yml` file was updated to remove the `phpstan` job. This change means we will no
longer run static analysis automatically on our CI pipeline. Further discussions may be needed on
whether to replace this with another tool or integrate it differently.

Modified 'var' declarations to 'const' in CouponCodeInput.js file for better ES6 compliance. This
refactoring helps increase code readability and ensures variables are block-scoped, reducing the
risk of accidental variable reassignment or mutation.

Debug test

In this commit, modifications have been made to the .gitlab-ci.yml file to allow running PHPUnit on
PHP 8.3 in our pipeline. A new job `modify-job` was added which runs phpstan script with following
instructions:

- Keeping other autoloaders with `QUIQQER_OTHER_AUTOLOADERS=KEEP`
- Disabling progress display with `--no-progress`
- Removing memory limit with `--memory-limit=-1`
- Enabling debug mode with `--debug`

The commit modifies the phpstan.dist.neon configuration file. The changes include removing the
excludePaths section and adjusting the end of the file to not have a new line. This simplification
will better suit our testing environment.

This commit includes a minor modification in the `phpstan.dist.neon` file where the `excludePaths`
value has been updated. Besides, a conditional check has been added in the `onPackageSetup`
function, under `Events.php` in the `src/QUI/ERP/Coupons` directory, to only proceed if the package
name is 'quiqqer/coupons'. This additional check generally improves the efficiency and correctness
of the package setup process.

This commit modifies the phpstan.dist.neon to exclude the DigitalCouponProductType.php file from
being analyzed by PHPStan. This was necessary due to consistent unresolvable errors.

This commit modifies the `src/QUI/ERP/Coupons/CouponCode.php` by removing an unexplained variable
annotation.
In `src/QUI/ERP/Coupons/Products/PhysicalCouponProductType.php`, it introduces an import statement
for `QUI\\Locale` and updated the `Locale` parameter type definition in `getTypeDescription`
function. This change improves code readability and consistency.

Reviewed code and made several improvements which provide better error handling, code quality, and
resolved warnings. The changes include:
- Syntax modifications for better readability,
- Ensured variable type checks before using methods,
- Resolved warning instances in phpstan,
- Adapted code to comply with modern PHP nullable type indicator.

Alongside, made version update in phive/phars.xml. Also, unnecessary comments or instances are
removed for a cleaner code.

Related: #16

Related: #17

Add QUIQQER Tooling

See merge request !13

Dev

See merge request !11

fix: quiqqer/quiqqer locale and paths adapted to quiqqer/core

See merge request !9

fix(CouponCodeInput): mobile buttons adjusted

See merge request !7

fix(CouponCodeInput): mobile buttons adjusted

See merge request !5

verfasst von Michael Danielczok vor 1 Jahr und Henning Leutz committete vor 10 Monaten

Related: quiqqer/template-cologne#103


(cherry picked from commit fe49e667)

fix: consider case in onQuiqqerOrderBasketRemovePos where article does not exist

See merge request !4

verfasst von Patrick Müller vor 1 Jahr und Henning Leutz committete vor 10 Monaten

(cherry picked from commit 017e2453)

feat!: quiqqer v2

See merge request !3