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.
Problem/Motivation
#2960136: Add the ability to use a prefix for all the tables in a table mapping was only committed to 8.7.x and TemporaryTableMapping
deprecation message points to 8.6.x
Proposed resolution
Fix the deprecation error message
Remaining tasks
Review patch
User interface changes
Non
API changes
Non
Data model changes
Non
Comment | File | Size | Author |
---|---|---|---|
#13 | 2991001-deprecation-message-13.patch | 2.02 KB | pasan.gamage |
Comments
Comment #2
pasan.gamage CreditAttribution: pasan.gamage at PreviousNext commentedComment #3
pasan.gamage CreditAttribution: pasan.gamage at PreviousNext commentedUploading the patch file per @jibran's comment on #2960136-27: Add the ability to use a prefix for all the tables in a table mapping
Comment #5
larowlanCrediting jibran who provided mentoring here
Comment #7
jibranCan you please make sure you are using 8.7.x branch as a base.
You have to replace it in three places.
@expectedDeprecation
message shouldn't be wrapped.Comment #8
pasan.gamage CreditAttribution: pasan.gamage at PreviousNext commentedRemoved wrapping.
Attaching new patchfile
Comment #9
pasan.gamage CreditAttribution: pasan.gamage at PreviousNext commentedPlease ignore patch file on comment #8
Attaching new patch file
Comment #12
jibranThis should be wrapped and you have to change the
@expectedDeprecation
ontestTemporaryTableMapping
not ontestGetFieldTableNameInvalid
Comment #13
pasan.gamage CreditAttribution: pasan.gamage at PreviousNext commentedComment #14
jibranLooks good now. Thanks!
Comment #15
alexpottCommitted b043354 and pushed to 8.7.x. Thanks!
Fixed the indentation in the comment on commit too.