Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
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.