Skip to content

Fix/ee admin order create crash final#68

Open
jules-paystack wants to merge 5 commits intomasterfrom
fix/ee-admin-order-create-crash-final
Open

Fix/ee admin order create crash final#68
jules-paystack wants to merge 5 commits intomasterfrom
fix/ee-admin-order-create-crash-final

Conversation

@jules-paystack
Copy link
Collaborator

No description provided.

… admin crash

The PaymentManagementInterface preference was registered in the global
etc/di.xml, making its implementation (PaymentManagement) resolvable in
all areas including admin. PaymentManagement depends on
Magento\Checkout\Model\Session — a frontend-only session class that
cannot be properly instantiated in the admin area.

On Adobe Commerce EE 2.4.8, the admin "Create New Order" page triggers
resolution of this dependency chain, causing a PHP exception that
crashes the entire page (6 MFTF test failures).

Changes:
- Move PaymentManagementInterface preference from global di.xml to
  etc/frontend/di.xml and etc/webapi_rest/di.xml
- Inject Checkout\Session via Proxy to defer instantiation until
  actually needed (defense in depth)
- Bump version to 3.0.5
…tests

ConfigProvider constructor eagerly called getMethodInstance() which crashed
the entire checkout page if it threw. Now deferred to getConfig() with
try-catch so the checkout page always renders. Same lazy-load pattern
applied to PaystackApiClient and AbstractPaystackStandard. Replaced
concrete Store model with StoreManagerInterface.
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