First of all, let me say "thanks!" This module is exactly what I am looking for

I am getting the following error when I try to enable this module:

Fatal error: Cannot redeclare anon_datereminder_allowed_access_node() (previously declared in /MYSITE/sites/all/modules/anon_datereminder/anon_datereminder.module:121) in /MYSITE/sites/all/modules/anon_datereminder/anon_datereminder.module on line 160

CommentFileSizeAuthor
#8 anon_datereminder-fater-error-upon-install-2283091-7.x-1.0.patch2.11 KBskadu

Comments

a214n’s picture

a214n commented

Also having this issue.. Can't enable module

bletch’s picture

bletch commented

Thanks for letting us know. I am away for the next week but will be able to take a look at this when I get back. It appears to be a fairly simple error - with that particular function anon_datereminder_allowed_access_node() being defined twice - not sure how that has changed but, again, I will take a look.

buttonwillowsix’s picture

buttonwillowsix commented

Thanks!

buttonwillowsix’s picture

buttonwillowsix commented

I changed the name of the second function, which allowed me to install it. Everything looks peachy, except that I don't see any actual reminder functionality on any of my event nodes! So, any advice here would be greatly appreciated.

webservant316’s picture

webservant316 commented

same problem observed here.

buttonwillowsix’s picture

buttonwillowsix commented

Any update on this? I see everything except the reminder button/link on my nodes.

FNGR’s picture

FNGR commented

same error here!

skadu’s picture

skadu
Primary language English
commented
StatusFileSize
new anon_datereminder-fater-error-upon-install-2283091-7.x-1.0.patch2.11 KB

Hi There,

This looks to be caused by a commit that had conflicts:

/**
 * Check if this user can access reminders for this node.
 *
 * @param node $node
 *   The node
 * @param string $acc
 *   What kind of access is requested?
 *   'all' means view others' reminders
 *
 * @return bool
 *   FALSE if user does not have access
 *
 * @ingroup callbacks
<<<<<<< HEAD
=======
 */
function anon_datereminder_allowed_access_node($node = NULL, $acc = 'all') {
  global $user;
  if (!isset($user)) {
    return FALSE;
  }

  module_load_include('inc', 'anon_datereminder', 'includes/defines');
  // First, be sure reminders are on for this.
  if (!isset($node)
    || ($node->anon_datereminder_enabled != ANON_DATEREMINDER_TYPE_ON)) {
    return FALSE;
  }

  if (user_access(ANON_DATEREMINDER_VIEW_OTHER_USER_REMINDERS)
    || user_access(ANON_DATEREMINDER_ADMINISTER_REMINDERS)) {
    return TRUE;
  }

  return (($acc == 'own') && (user_access(ANON_DATEREMINDER_REQUEST_REMINDER)));
}

/**
 * Implements hook_nodeapi().
>>>>>>> 13649003a161dde9cb04a8419f78647e5bbf1ad3
 */

By removing the first of these duplicated functions (pasted above) the module appears to install, I did not test the whole module, as it won't actually work in my use case, but I wanted to at least put a patch up here so you can get it installed.

See attached.

  • bletch committed fc8ac3f on 7.x-1.x 
    Issue #2283091 by bletch: removed merge conflict
bletch’s picture

bletch commented
Status: Active » Fixed

Thanks skadu for the patch. I thought this was taken care of a long time ago. Have updated repo.

bletch’s picture

bletch commented
Status: Fixed » Closed (fixed)