Closed (fixed)
Project:
Commerce License
Version:
8.x-2.0-alpha9
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Apr 2018 at 05:17 UTC
Updated:
17 May 2018 at 08:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
joachim commentedYup, I realized after I made the release I should have taken those out too!
Don't suppose you could post a patch of them commented out?
Comment #3
fy1128 commentedHere is the patch.
Comment #5
joachim commentedThanks!
Comment #6
nikathoneRemoving the entire
commerce_license.links.action.ymlCause gettingComment #7
joachim commentedDo you maybe have a nonstandard setup? Because I did 'drush cr' with the latest release and it all worked fine.
Comment #8
joachim commentedI just did a drush cr on a site that's running the latest dev, and the licenses page loads fine for me.
Comment #9
nikathoneHere is my stack using drupalvm:
Also using php7.0 with relocated doc root on pantheon it's breaking. Is there any particular reason we should keep a file which is not used at all?
Comment #10
chriso commentedWe (Acquia) have just seen a customer hit the same problem with 8.x-2.0-alpha10 - corrected by the patch in #6.
Acquia Cloud uses the PECL YAML extension. We have a snippet for locating potential sources of the yaml_parse() exceptions on the following article, which helped us locate this issue: https://docs.acquia.com/article/yaml-pecl-serialization-errors
Some background on why this might affect some and not others is here: #1920902: Add a Drupal Yaml wrapper so we can default to PECL Yaml component if it is available
Comment #11
nikathoneLet put this back in needs review...
Comment #13
joachim commentedThanks @chriso for the explanation and @nikathone for the patch!