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'm using site-verify with an installation profile, and it'd be nice to have API functions to add new verifications.
This would require refactoring of the queries that record the verifications from the form-submit handlers:
drupal_write_record('site_verify', $form_state['values'], array('svid'));
and moving those into API functions.
Comment | File | Size | Author |
---|---|---|---|
#2 | site_verify-1059798-use_API_functions.patch | 10.9 KB | manarth |
Comments
Comment #1
Dave ReidYeah definitely. Going to add it after the 1.0 version.
Comment #2
manarth CreditAttribution: manarth commentedHere's a patch based on head.
The 2 new API functions are:
- site_verify_save($verification);
- site_verify_delete($svid);
The form handlers use the API functions, and there are a few new code-comments.
Comment #3
joachim CreditAttribution: joachim commentedPatch has a lot of unrelated changes, some of which look like they would have been part of a D7 upgrade.
Comment #4
joachim CreditAttribution: joachim commentedIt also no longer applies cleanly...
However, the patch contains has a lot of documentation improvements, which I'll commit now, as these are always a good thing. Thanks!
git commit -m "by manarth: Fixed missing function documentation; added comments to code." --author="manarth "
Commit b9f12c18835398e92896b7c81c40f250b4985055.