Doesn't work on new version of Drupal

Subscribe

I tried to make a patch, it seems to work for me.

I also fixed some drupal coding standard issues that I found.

Patch worked great for me, Thanks :-)

Hi @Ie72,

no there are many rows affected by the fix:

-    	'arguments' => array('display' => NULL),
+    	'variables' => array('messages_drupal' => array()),

-    	'arguments' => array('content' => NULL),
+    	'variables' => array('content' => NULL),

-	$message = drupal_get_messages($display['display'], FALSE);
+	$message = drupal_get_messages($display['display']);

-		$message = theme('better_messages_content', $display['display']);
+		$message = theme('better_messages_content', array('messages_drupal' => $message));

-	else { $output = theme('better_messages_content', $display['display']);}
+	else {
+  	$output = theme('better_messages_content', array('messages_drupal' => $message));
+  }

-function theme_better_messages_content($display = NULL) {
+function theme_better_messages_content($variables) {

-	foreach (drupal_get_messages($display) as $type => $messages) {
+	foreach ($messages_drupal as $type => $messages) {

Given that the module simply does not appear to work with Drupal 7.33 (i.e. actual messages are not displayed at all, and a blank message box is displayed) and has no workaround, this seems like a critical issue, rather than major, no?

The patch worked but the theme of the box is not the same. The rounded corners are gone. Any CSS fixes for this ?

@brainHax my theme is still the same, with rounded corner. Maybe it's an issue with your theme, and in any case not related to this issue I guess.

@mecmartini
Thank you for your patch. It's working great!

Thanks for your patch @mecmartini , It's working.

Thanks a ton for this - patch from #2 worked great - nice work!

Thank you mecmartini!