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.
I enabled maxlength so one of my fields could benefit from its use. All of the sudden, all of my features have maxlength settings injected into them when I re-export. IMHO this is a major issue, this module is essentially unusable for major sites where they expect clean settings exports.
Shouldn't it work like this:
1) On edit form, user sees ability to enable the maxlength module for this text area
1a) if that checkbox not checked, no processing is done on that field
2) user checks box, only after that are maxlength hooks added to that field
Comment | File | Size | Author |
---|---|---|---|
#2 | 1867566-maxlength-settings-added-in-incorrect-fields.patch | 1.83 KB | slv_ |
Comments
Comment #1
joelcollinsdc CreditAttribution: joelcollinsdc commentedFor example, this is the diff of a content type (that i didn't enable maxlength on at all)
Comment #1.0
joelcollinsdc CreditAttribution: joelcollinsdc commentedbeing less of a jerk
Comment #2
slv_ CreditAttribution: slv_ commentedPatch attached. That should fix the issue.
Comment #3
joelcollinsdc CreditAttribution: joelcollinsdc commentedThanks!
Comment #4
lpalgarvio CreditAttribution: lpalgarvio commentedhow is this coming along?
Comment #5
lpalgarvio CreditAttribution: lpalgarvio commentedI've tested it and it works fine. May be ready for commit and roll out on new release
Comment #6
lpalgarvio CreditAttribution: lpalgarvio commentedstill no issues so far. please commit
Comment #7
B-Prod CreditAttribution: B-Prod commentedPatch in #2 fixes the issue, commit would be welcome.
Comment #9
frjo CreditAttribution: frjo commented