Hi

After installing LimeSurvey_sync I lost the Import tab for Mailhandler Source nodes associted with feeds importers. Found the code below to be causing the Import menu item to be sharing space with the Synchronize menu item (both of which cause a tab to show on a node). I renamed node/%node/import to node/%node/import1 in feeds.module to solve the issue, but a better solution would seem in order

Thanks

In feeds.module there is:

function feeds_menu() {
  $items = array();
...
  $items['node/%node/import'] = array(
    'title' => 'Import',
    'page callback' => 'drupal_get_form',
    'page arguments' => array('feeds_import_tab_form', 1),
    'access callback' => 'feeds_access',
    'access arguments' => array('import', 1),
    'file' => 'feeds.pages.inc',
    'type' => MENU_LOCAL_TASK,
    'weight' => 10,
  );

and in ls_survey.module there is

function ls_survey_menu() {
  $items = array();
...
  $items['node/%node/import'] = array(
    'title' => 'Synchronization',
    'page callback' => 'drupal_get_form',
    'page arguments' => array('ls_survey_import', 1),
    'access callback' => 'ls_survey_import_access',
    'access arguments' => array(1),
    'type' => MENU_LOCAL_TASK,
    'weight' => 15,
    'file' => 'ls_survey.import.inc',
  );
CommentFileSizeAuthor
#1 fix_feeds_compatibility-2177819-1.patch4.42 KBthedut
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thedut’s picture

Assigned: Unassigned » thedut
Status: Active » Patch (to be ported)
FileSize
4.42 KB

Hello hanksterr7,

Thank you for reporting this issue. It only occurs when the Feeds module is enabled.
I will include this fix into the next LimeSurvey Sync release.

thedut’s picture

Status: Patch (to be ported) » Closed (fixed)

Fixed into the LimeSurvey Sync 7.x-1.6 release and 6.x-1.6 release.