in function quicktabs_export_form()

  // Generate export code
  $code = '$items = array();' ."\n";
  $code .= ctools_export_object('quicktabs', $qt, '');
  $code .= '$items["'. $qt->machine_name .'"] = $quicktabs;' ."\n";
  $code .= 'return $items;';

should be just:

  // Generate export code
  $code = ctools_export_object('quicktabs', $qt, '');

Why:
- views module does the same, it only makes the export code of 1 view, not an array of views.
- array of quicktabs can be generated with ctools bulk_export

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pasqualle’s picture

Issue tags: +Novice

.

barbi’s picture

Assigned: Unassigned » barbi
barbi’s picture

Status: Active » Needs review
FileSize
951 bytes
valthebald’s picture

Status: Needs review » Reviewed & tested by the community
  1. Patch still applies to the latest 7.x-3.x
  2. Output of export form without patch differs from views export format
  3. Output of export form with patch has format similar to views export
  4. I don't know what was the reason for including $items array in export, but patched behavior makes more sense to me, so marking RTBC.
ultimateboy’s picture

Status: Reviewed & tested by the community » Fixed

Thanks. Committed to 7.x-3.x

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