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.
When adding allowed sources, it limits you to 128 characters. I need to add more allowed sources and that limit is keeping me from doing so. Feel like that limit is too constrictive and needs to be updated to allow more. Might be better and easier to read to convert those fields to a textarea so their not limited (https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.h...).
Forgive me if I did anything wrong here, first patch I've ever committed and fairly new to Drupal.
Comment | File | Size | Author |
---|---|---|---|
#10 | seckit-increase-textfield-length-2150731-10-D6.patch | 4.73 KB | milodesc |
#2 | 2150731-increase-textfield-length.patch | 5.05 KB | wiifm |
Comments
Comment #1
bmarshall CreditAttribution: bmarshall commentedComment #2
wiifmHad the same issue, I do however think that a textarea will make it hard to author, as undoubtedly someone will place a newline character in there. Attached is a new patch that simply increases the maxlength of the field to 1024 characters (which I think will fit everyone's needs).
Comment #3
wiifmAlso changing this to a bug report as this is actively preventing me from using the correct rules as they do not fit
Comment #4
jweowu CreditAttribution: jweowu commentedAgreed. Committed #2 in 61339ac0db7c05aaaf5b0c71c7fbc1663ab3aa2c.
Comment #6
jweowu CreditAttribution: jweowu commentedIncluded in 7.x-1.9-rc1.
Please follow #2200671: 7.x-1.9 release candidate to track any additional release candidates prior to the stable release.
Comment #8
milodescFor what it's worth, #2 applied cleanly to 6.x-1.7.
Comment #9
jweowu CreditAttribution: jweowu commentedThanks milodesc, but I see:
Comment #10
milodescWhoops. You're right. There was a difference between the D6 and D7 versions of the module in the '#description' value for $form['seckit_xss']['csp']['report-uri'] that was throwing off the patch application.
The new attached patch should work.
Comment #11
jweowu CreditAttribution: jweowu commentedCommitted and pushed to 6.x-1.x; thanks!