I'm not sure if this is related to http://drupal.org/node/35588 as I am not running PHP 5. I am having a problem with the admin page www.mydomain.com/subdirectory/?q=admin/settings/project/status is coming up blank. There are no errors in the log, so I don't have any more info at this point.

I did have a problem with a nodewords conflict that was generating errors when responding to issues (http://drupal.org/node/39500) but that was rectified by associating a vocabulary with project and issue nodes -- I mention it here only on the odd chance that it's related.

The bug is marked "critical" only because an entire admin section is missing. For myself, the module functions well enough for our purposes. Sorry I don't have more info.

CommentFileSizeAuthor
#3 project_issue-blank_settings.patch796 bytesdww
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

konst’s picture

It seems that you can cure this problem by replacing "return $output" with "print theme('page', $output)" in a pair of places in issue.inc:

--- issue.inc.old       2005-07-20 04:02:48.000000000 +0300
+++ issue.inc   2006-01-28 16:24:54.000000000 +0200
@@ -625,11 +625,11 @@
         array('data' => t('Default')),
         array('data' => t('Operations'))
       );
-
+
       $output = '<div>' . theme('table', $header, project_issue_admin_states()) . '</div>';
       $output .= form_submit(t('Save'));
-
-      return form($output);
+      // XXX: Changed this: return form($output); to this:
+      print theme('page', form($output));
       break;
   }
 }
@@ -699,7 +699,8 @@
       t('Delete'),
       t('Cancel'),
       $extra);
-    return $output;
+    // XXX: Changed this: return $output; to this:
+    print theme('page', $output);
   }
 }
tgorg.com’s picture

Title: Blank page on admin/settings/project/status » Works Here...

I've inserted the code into my drupal site and the problem was fixed...

dww’s picture

Title: Works Here... » Blank page on admin/settings/project/status
Status: Active » Needs review
FileSize
796 bytes

setting the title back to something meaningful. ;)

also, i noticed that this is duplicate with another patch i submitted.
however, i notice that your version fixes this problem in the project_issue_admin_delete_state() function, too.

so, i'm going to set that other issue to "duplicate" and include a patch here that fixes both.

this issue is now pending review. i've got cvs access, so i'd be happy to commit it, i just need to hear from a project maintainer that it's ok to do so.

thanks,
-derek

laura s’s picture

I can confirm that this patch addresses the issue I reported. Thanks!

nedjo’s picture

Please go ahead and apply this.

dww’s picture

Status: Needs review » Fixed

fix applied to 4.6. i verified this isn't a problem in HEAD.

dww’s picture

Assigned: Unassigned » dww

assigning this issue to myself (i guess i should have done that when i started working on the patch). ;)

Anonymous’s picture

Status: Fixed » Closed (fixed)