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.
As a follow-up for #2157783: Deploy_7 code has an unused old_uri
The D6 code does some rewriting of table data in platform/drupal/deploy.inc. While the D7 code only sets three file_*_path variables...
Comment | File | Size | Author |
---|---|---|---|
#8 | provision-code-2157785-8.patch | 2.76 KB | helmo |
#3 | provision-code-2157785-3.patch | 2.1 KB | helmo |
Comments
Comment #1
helmo CreditAttribution: helmo commentedComment #2
helmo CreditAttribution: helmo commentedTss ... the three file_*_path variables don't get used anyway as we write these values to the settings.php
So for D7 this is at the moment effectively a NO-OP.
Comment #3
helmo CreditAttribution: helmo commentedHere's a patch to review... with todo's :)
Comment #4
ergonlogicIn deploy.inc, we also support deploying from a site that had been installed in 'default'. Could we do that here too?
So... we don't need these anymore?
Comment #5
helmo CreditAttribution: helmo commentedYes, I'll thought about adding that. But I hoped to get the "get Field API to tell us all text fields and loop over the tables" todo resolved first.
Yes, I'll make that a separate commit to describe that these are already written to the settings.php. --> done fcec4d3a0beb9f73ecded28fdc2fcd582f9f4c65
I'm hesitant to commit the rest of this issue in such a late release candidate stage...
Comment #6
ergonlogicThis looks more like a bug to me, since this functionality already exists for d6.
Comment #7
helmo CreditAttribution: helmo commentedznerol mentioned in irc that https://api.drupal.org/api/drupal/modules%21field%21field.info.inc/funct... could help solve the todo "get Field API to tell us all text fields and loop over the tables ?"
Comment #8
helmo CreditAttribution: helmo commentedHere's an updated patch.
* updated to D7 DB API
* re-added sites/default support
I've opened #2163979: Use Field API in deploy_7 code to rewrite in all content tables as a follow-up.
Comment #9
helmo CreditAttribution: helmo commentedI committed this to 7.x-3.x after some more testing, and included #2163979: Use Field API in deploy_7 code to rewrite in all content tables
commit: 3d0d14a6702509efd470282d3450f682670e8122
I think this can also be committed to the 6.x-2.x branch, I've started the branch 6.x-2.x-backports to prepare this.
Comment #11
ergonlogicThis needs to get into 6.x-2.1
Comment #12
anarcat CreditAttribution: anarcat commentedWhich means it's a critical.
Comment #13
helmo CreditAttribution: helmo commentedMerged into 6.x-2.x