Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
update.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Mar 2015 at 17:35 UTC
Updated:
22 Apr 2015 at 17:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
a_thakur commentedPlease find the attached patch which fixes the issue.
Comment #2
jeroentPatch looks good, but we still need a change record.
Comment #3
mile23The test passed on March 7. Adding 'needs change record' tag though I'm not sure that's really necessary.
Retesting the patch in #1, which should change RTBC to Needs Work if it fails.
Patch removes
update_project_storage()and useages (there's only one).Thanks, folks!
Comment #5
alexpottThis just be replaced not removed. With an @see to the projectStorage method.
Comment #6
jeroentAdded reference to the projectStorage method.
Patch attached.
Comment #7
jeroentComment #8
mile23Still applies, does change the @see from update_project_storage() to \Drupal\update\UpdateManagerInterface::projectStorage().
Comment #9
alexpottStill need to attach a CR to this - maybe there is an existing one.
Comment #10
mile23Added change to existing record for
update_*()functions.: https://www.drupal.org/node/2430443Comment #11
a_thakur commentedThanks Mile23 for the change record.
Comment #12
webchickOk, looks like this is in order now. Removing deprecated functions in explicitly allowed in beta.
Committed and pushed to 8.0.x. Thanks!
Comment #14
a_thakur commented