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.
The module should really use the File API instead of storing file details in {site_verify}. It can save its uploaded files to sites/default/files/site_verify
It should even be possible to backport this to Drupal 6 as well.
Comment | File | Size | Author |
---|---|---|---|
#7 | use-file-api-696436-7.patch | 12.25 KB | lathan |
#5 | use-file-api-696436-5.patch | 11.96 KB | lathan |
#4 | use-file-api-696436-4.patch | 11.59 KB | lathan |
#3 | use-file-api-696436-3.patch | 11.72 KB | lathan |
#1 | use-file-api-696436-1.patch | 10.76 KB | lathan |
Comments
Comment #1
lathanHere is a patch to do this.
Comment #2
lathanComment #3
lathanI have rerolled this patch with the addition of #1391860: Add validation for meta tags as they conflict.
Comment #4
lathanFixed white space issue.
Comment #5
lathanLast time, fixed issue when merged patches.
Comment #6
mrfelton CreditAttribution: mrfelton commentedI think this is missing
db_drop_field('site_verify', 'file');
. The schema module is reporting "file: unexpected column in database".Comment #7
lathanIssues fixed.
- fixed meta verify was on wrong var.
- install schema fixed as pointed out by @mrfelton
- removed unneeded meta output at url.
Comment #8
lathanComment #9
joachim CreditAttribution: joachim commentedPatch no longer applies cleanly I'm afraid.
A few things that need work too:
Some blank lines are needed here to conform to docs standards.
'Prepare'?
What happens if I have not configured a private directory?
This is part of the patch at #1391860: Add validation for meta tags.
This looks like an unrelated change, at least partly. Why change from drw() to db_update()?
s/file/field/ ?
I don't follow this change.
Comment #10
joachim CreditAttribution: joachim commentedAlso, compare with #1230956: Support for Exportables (Features), which is (I think) incompatible with the direction taken here.