Update hook implementations to Drupal 7 [#2625566]

The hooks permission and nodeapi are not valid for drupal 7. This effects the following code bocks

/**
* Implements hook_perm().
*/
function webform_to_gdocs_perm() {
return array('administer webform_to_gdocs', 'attach webform to gdocs');
}

/**
* Implements hook_nodeapi().
*/
function webform_to_gdocs_nodeapi(&$node, $op) {
// Delete any webform_to_gdocs references to this node.
if ($op == 'delete') {
db_query("DELETE FROM {webform_to_gdocs_webforms} WHERE nid = %d", $node->nid);
}
}

Comment	File	Size	Author
#9	interdiff-8-9.txt	925 bytes	jwilson3
#9	webform_to_gdocs-update_d6_hooks-2625566-9-D7.patch	1.26 KB	jwilson3
#8	webform_to_gdocs-update_d6_hooks-2625566-8-D7.patch	1.26 KB	progpapa
#8	webform_to_gdocs-update_d6_hooks-2625566-8-D7.patch	1.26 KB	progpapa
#7	drupal7.patch	1.25 KB	richgerdes
#4	Update-Hook-Permission-2625566-4.patch	700 bytes	joecorall

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

30 November 2015 at 07:55

miguelrro created an issue. See original summary.

Comment #2

charlie-s CreditAttribution: charlie-s commented 30 November 2015 at 15:35

Are you positive you're on version 4.2, have the module installed, and are at admin/config/content/webform/gdocs? What do you see at that page - a 404 message? A blank screen? Something else?

Comment #3

joaorodr84 CreditAttribution: joaorodr84 commented 31 January 2016 at 22:13

Hi.

I am using Drupal 7, with Webform 7.x-4.12 and Webform to Google Docs 7.x-4.2. Both modules are activated.

Like @miguellro, I have succesfully performed the instructions from point 1.

But at point 2, the link "[my_website]/admin/config/content/webform/gdocs" does not work.

I get this:
A página pedida "/admin/config/content/webform/gdocs" não foi encontrada.

Which means:
The requested page "/admin/config/content/webform/gdocs" could not be found.

How should I proceed?

Thanks a lot.

Comment #4

joecorall CreditAttribution: joecorall commented 8 February 2016 at 14:31

Status:

Active

» Patch (to be ported)

File	Size
Update-Hook-Permission-2625566-4.patch	700 bytes

Hello,

This module's permission definitions were using the D6 version of the hook. The attached patch makes the permissions available for D7.

After applying this patch, you'll need to go to /admin/people/permissions and give the respective roles the "Administer Webform to Google Docs" permission.

Comment #5

ghedolo CreditAttribution: ghedolo commented 8 February 2016 at 18:50

tested patch on drupal 7: it seems does not work....

Comment #6

joecorall CreditAttribution: joecorall commented 8 February 2016 at 19:46

@ghedolo Do you see the two permissions at /admin/people/permissions (example screenshot of what you should see)?

If not, you will need to clear the cache at /admin/config/development/performance and try /admin/people/permissions again.

Comment #7

richgerdes

he/him

English

New Jersey, USA

CreditAttribution: richgerdes commented 10 November 2016 at 18:34

Title:	Config tab does not show up	» Update hooks to Druapl 7
Version:	7.x-4.2	» 7.x-4.x-dev
Component:	User interface	» Code
Category:	Support request	» Bug report
Priority:	Major	» Critical
Issue summary:	View changes
Status:	Patch (to be ported)	» Reviewed & tested by the community

File	Size
drupal7.patch	1.25 KB

1 file was hidden/shown/deleted

File	Size
Update-Hook-Permission-2625566-4.patch	700 bytes

Changes to the summary and updated it to reflect the real issue here. Rebuilt the patch to be a better solution to the problem as it did not only effect just one function.