Replace 'event' by 'operation' in all modules

kbahey - June 2, 2008 - 19:46
Project:User Points Contributed modules
Version:5.x-3.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

This is a parallel issue to http://drupal.org/node/265714.

The attached patch changes all occurrences of 'event' to 'operation' in the modules to be consistent with the API.

AttachmentSize
event-to-operation.patch16.29 KB

#1

Kiam@avpnet.org - June 2, 2008 - 20:29
Status:patch (code needs review)» patch (reviewed & tested by the community)

I tested it.

I would suggest to write links to issues by putting the issue number between [] like in [#265714]; in that way it is possible to know if the issue is not active anymore (and to know which issue is, as Drupal puts the title of the issue).
For the example above, Drupal outputs #265714: Rename 'event' parameter in userpoints_userpointsapi() to 'operation'.

#2

jredding - June 3, 2008 - 02:35

Looks good, this is an overlooked bug. The hook should've also been called operation.

#3

kbahey - June 3, 2008 - 03:08
Version:5.x-3.x-dev» 5.x-3.0
Status:patch (reviewed & tested by the community)» fixed

Committed.

#4

Anonymous (not verified) - June 17, 2008 - 03:12
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.