Error messages are not being displayed in "Site Configuration/phpBB2Drupal Settings" when something is wrong. I looked at the code and found the $output was being overwritten in phpbb2drupal.pages.inc on line 454 ( $output = should be $output .= ).

Also, '#collapsed' should be set to FALSE for either case, since an error or the ready message should be displayed.

Lines 449-455 should read:

  $form['migration']['#collapsed']= FALSE;
  if ($ready_for_migration) {
    $output .= '<p>'. t('Check again all the settings above, and if everything is ok, <a href="@migratelink">you can now proceed with the migration</a>', array('@migratelink' => url('admin/phpbb2drupal/migrate'))) .'</p>';
  }
  else {
    $output .= '<p class="marker">'. t('You are not ready for the migration yet. Check the settings above.') .'</p>';
  }

Comments

naheemsays’s picture

Status: Active » Closed (fixed)

This should be fixed.