Problem/Motivation

Drupal\Core\Config\UnmetDependenciesException: Configuration objects provided by <em     [error]
class="placeholder">commerce_pos</em> have unmet dependencies: <em
class="placeholder">field.field.commerce_order.pos.order_items
(field.storage.commerce_order.order_items)</em> in

I checked on my install of commerce 2.x (Dec 31, 2019) and there is no field.storage.commerce_order.order_items.yml file in that config

Proposed resolution

Commerce has moved

  • order_items on the commerce order entity to a base field
  • stores on the commerce product entity to a base field

Therefore we no longer need to provide configuration for these.

This makes the latest version of this module dependent on commerce 2.16

Remaining tasks

User interface changes

None

API changes

None

Data model changes

Less configuration.

Release notes snippet

The latest version of this module is dependent on commerce 2.16

Comments

SocialNicheGuru created an issue. See original summary.

derekcresswell’s picture

Status: Active » Closed (duplicate)
Related issues: +#3032258: Remove use of Profile 1.x-dev in drupalci.yml

The issue of field.storage.commerce_order.order_items.yml being an unmet dependency is being addressed in a different issue here #3032258.
Thank you.

derekcresswell’s picture

Status: Closed (duplicate) » Active
derekcresswell’s picture

StatusFileSize
new1.37 KB

This should remove the unmet dependencies warnings.
And as I thought, the next main issue here is that the pos index is in the optional config. Personally unsure how drupalci sets up all that stuff but will look into. Help appreciated.

derekcresswell’s picture

StatusFileSize
new2.04 KB

This does not fix everything but I would like to leave it here as it does help some.
The UPCTest in the kernel tests of POS was out of date. See this commerce issue

derekcresswell’s picture

StatusFileSize
new3.5 KB

Just trying something more so. The indexItems on null should be solved by the changes to server config.
The Ajax change will likely be done to all in the future.

derekcresswell’s picture

socialnicheguru’s picture

Status: Active » Needs review
derekcresswell’s picture

StatusFileSize
new5.94 KB

ATM I am not aware how to fix the config problems with order_types I'll get that next time.

Status: Needs review » Needs work

The last submitted patch, 9: 3104902-9.patch, failed testing. View results

alexpott’s picture

Version: 8.x-2.5 » 8.x-2.x-dev
Status: Needs work » Needs review
StatusFileSize
new1.89 KB
new7.43 KB

Fixing the last test. We need to declare a dependency on commerce_order 2.16 because that moves order_items to a base field which has big changes on this module's configuration.

alexpott’s picture

Title: Please update to be compatible with commerce 2.16 » Update to be compatible with Commerce 2.16
Issue summary: View changes
alexpott’s picture

Status: Needs review » Reviewed & tested by the community

All green - let's unblock all the other issues as tests on HEAD are broken.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

  • alexpott committed 46ffc1d on 8.x-2.x
    Issue #3104902 by DerekCresswell, alexpott: Update to be compatible with...
alexpott’s picture

I've opened #3111635: Duplicate dependencies for the funky search api config.

  • alexpott committed f5ac32c on 8.x-2.x
    Issue #3104902 by DerekCresswell, alexpott: Update to be compatible with...

  • alexpott committed d458739 on 8.x-2.x
    Issue #3104902 by DerekCresswell, alexpott: Update to be compatible with...
alexpott’s picture

Had to remove the drupal constraint because there are so many times when Drupal has no idea of the module version - for example when DrupalCI gets commerce using composer with --prefer-source

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.