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.
Since Site Verification stores its data in its own table {site_verify}, it isn't possible to add them to a Feature module.
Would be very useful to be able to do so on an individual basis so that each entry can be put in different feature modules when so is required.
Comment | File | Size | Author |
---|---|---|---|
#14 | 1230956-13-features_exportables.patch | 5.82 KB | netw3rker |
#11 | exportables-1230956-11.patch | 20.66 KB | realityloop |
#6 | interdiff.txt | 1.33 KB | Deciphered |
#5 | exportables-1230956-4.patch | 31.22 KB | Deciphered |
Comments
Comment #1
jdleonardI am also interested in this feature. Thanks!
Comment #2
tobias.grasse CreditAttribution: tobias.grasse commentedCount me in.
Comment #3
Deciphered CreditAttribution: Deciphered commentedI've begun work on this and will supply a patch ASAP.
It is being done using CTools with the CTools Export UI which means it isn't a minor task, but it's really the only logical way to do it.
Comment #4
Deciphered CreditAttribution: Deciphered commentedPatch attached (sponsored by CustomFormatters.com).
The only thing that currently isn't working is the 'Delete' portion of the simpletest, all the functionality works as expected and an upgrade path is included.
Comment #5
Deciphered CreditAttribution: Deciphered commentedOops, helps if I attach the patch...
Comment #6
Deciphered CreditAttribution: Deciphered commentedUpdated patch for a minor issue with the disabled status flag.
Comment #7
indytechcook CreditAttribution: indytechcook commentedThe features integration works fine and so does the UI. I didn't do tons of testing so I can't mark RTBC but it probably is.
@deciphered, for future reference, you can have ctools exportables without using the ctools exportables UI. Great work!
Comment #8
Deciphered CreditAttribution: Deciphered commentedI'm aware of that, but CTools Export UI means we don't have to re-invent the wheel, so why not take advantage of it.
Comment #9
Deciphered CreditAttribution: Deciphered commentedAs per #2136309: Upgrade path issue, which I accidentally opened not identifying this patch as the issue, the upgrade path needs work. Which this patch does.
Comment #10
joachim CreditAttribution: joachim commentedPatch looks very comprehensive -- tests too! :)
But also huge and scary, especially as I'm a temporary maintainer who's not familiar with the module code. So probably not one for me to deal with, sorry.
I think this is going to clash with #696436: Use File API and file_transfer(). In particular, am I right in guessing that when you upload a file to be a verification file, its contents get put into the table? That's completely at odds with what #696436: Use File API and file_transfer() is seeking to do -- and furthermore, using File API would actively prevent verification items from being exportable! So there's a question to be resolved about the direction this module should take. Which again, is not one for me, sorry!
Here's a review with a few things that need work:
Could we have comments here to explain what the ajax does? When I select an engine, I just get the AJAX spinner and then the exact same form comes back.
This needs documenting.
Comment and code do not agree.
This hook_update_N() is numbered incorrectly. It should be 7001.
Quite a few of these unrelated whitespace changes in this patch.
Comment #11
realityloopRerolled for current dev version
Comment #12
netw3rker CreditAttribution: netw3rker commented@realityloop,
I tried this patch & it blew away my ability to get back to http://example.com/admin/config/search/verifications is that by design? If so, what's the new admin URL?
Comment #13
netw3rker CreditAttribution: netw3rker commentedIn reviewing the work from @realityloop's patch, it looks to be a pretty extensive overhaul of the site_verify module in order to support the full ctools exportables framework. This might be overkill for what's need in order to meet the goals of the issue (specifically: providing features support for this module).
I've gone ahead and implemented the specific code required just for features support. this involves simply adding the export value to the db_schema, and then implementing the features_render and features_revert hooks.
The only downside to the attached patch is that it relies on "engine" as the key name, so you can't have more than one verification with the exact same engine name. That should be resolvable with the "custom" option though.
Anyhow, sorry @realityloop for the competing patch option, but this much less impactful to the current versions of the module.
Comment #14
netw3rker CreditAttribution: netw3rker commentedTurns out a machine name is kinda important to the function of this :). I've added a machine name field, and an update hook to populate it. the default machine name for existing entries will be [engine]_[svid]. that can be change before export in the UI by editing the machine name.
Comment #15
carolpettirossi CreditAttribution: carolpettirossi at CI&T for Pfizer, Inc. commented#13 works for me.
#14 does not work.
Comment #16
realityloop@netw3rker. I was just re-rolling an earlier patch.. no harm or foul. :)