Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The comments for the votingapi_set_votes function indicate that the content_id and fields are required and all other values are optional. I wrote a unit test that shows that the content_type is also a required field. (see the unit test patch at http://drupal.org/node/284010).
Obviously there is no current use in which the content_type is not being specified, so I recommend that we simply change the documentation to match the current behavior.
| Comment | File | Size | Author |
|---|---|---|---|
| votingapi_doc.patch | 1.1 KB | paul.lovvik |
Comments
Comment #1
paul.lovvik commentedComment #2
eaton commentedActually... I changed it so that it assumes nodes if no type is passed in, to match the documentation. Thanks for the simpletests, by the way: they're now part of the votingapi package, and helped quite a bit in tracking down a couple of obscure problems!
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.