This functionality is duplicated in merci_ui as a Block driven by a View. I think this should be removed from merci.module since the merci_ui approach is more flexible.

if (user_access("administer MERCI") || user_access("manage MERCI")) {
      drupal_set_message(t('Warning: @title checked out by <a href="@reservation_link">@reservation_title</a> which was due back @date', array('@title' => $node->title, '@reservation_link' => url('node' . $node->reservation_nid), '@reservation_title' => $node->reservation_title, '@date' => $node->field_merci_date_value2)));
    }

Comments

kreynen’s picture

Status: Active » Needs review

I believe this has already been removed.

kreynen’s picture

Title: Remove Error » Remove Redundant Error Message
darrick’s picture

Status: Needs review » Needs work

This has been removed. But I'm also working on a update which moves this to merci_validate_merci_selected_items. Right now if there is a conflict for a reserved item the grid shows even if the conflict is from an item not checked back in.

darrick’s picture

Title: Remove Redundant Error Message » Handle conflicts due to late returns in conflict_grid properly
darrick’s picture

Status: Needs work » Needs review
darrick’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)
Issue tags: -merci_rc

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