Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
From #1850418: Provide D7 -> D8 Views upgrade path in contrib, not core. The Views D7 -> D8 upgrade path will be provided by http://drupal.org/project/views_d8_upgrade, which will migrate the user's data and uninstall the D7 version of Views so that the D8 core version of Views can be installed cleanly. So, we should prevent the user from running update.php if a pre-D8 version of Views is installed.
Attached is not testable until #1848998: Problems with update_module_enable() is resolved.
Comment | File | Size | Author |
---|---|---|---|
#21 | vdc-1851386-21.patch | 1.5 KB | dawehner |
#21 | interdiff.txt | 1.3 KB | dawehner |
#15 | drupal-views-requirements-1851386-15.patch | 3.27 KB | kmoll |
#10 | 1851386-views-requirements.patch | 3.27 KB | kmoll |
views-requirements.patch | 1.54 KB | xjm | |
Comments
Comment #1
xjmComment #2
dawehnerAdd tag. On one hand this would be super-usefull for the users, though i ask myself, whether we can link to contrib projects out of core?
Comment #3
xjmIn general I'd say no, but in this case I think we should. This one hook implementation would be the only reference to contrib Views we'd have, and doing it this way guarantees it's the only one we'll need or that users will expect.
Comment #4
xjmComment #5
dawehnerviews-requirements.patch queued for re-testing.
Comment #6
dawehnerThis would be indeed a nice patch!
Comment #8
dawehner.
Comment #9
kmoll CreditAttribution: kmoll commentedworking on re-roll
Comment #10
kmoll CreditAttribution: kmoll commentedWhen re-rolling the patch i got conflicts with functions that don't seem to be in the latest version, so I removed them and recreated the patch. Those functions were views_update_last_removed(), views_update_8000, views_update_8001(), views_update_8002
Comment #11
kmoll CreditAttribution: kmoll commentedComment #13
kmoll CreditAttribution: kmoll commented#10: 1851386-views-requirements.patch queued for re-testing.
Comment #15
kmoll CreditAttribution: kmoll commentedfixed and recreated patch
Comment #16
kmoll CreditAttribution: kmoll commentedComment #18
kmoll CreditAttribution: kmoll commented#15: drupal-views-requirements-1851386-15.patch queued for re-testing.
Comment #20
kmoll CreditAttribution: kmoll commentedComment #21
dawehnerget_t() is already replace by the translator service.
Comment #33
quietone CreditAttribution: quietone at PreviousNext commentedNo need for this.
Thanks for everyone who contributed!