Port Better Messages to D7

subscribe

The official Drupal 7 release is out now and it would be really cool to have this awesome module on board.
To speak for myself: I use it on EVERY of my projects. :)
Thanks a lot for all your work on this module. I love it! :)

suscribing

subscribing. may help in porting but hardly busy with advupload in making features and preparing d7 port

Subscribe

Porting has started. Will have a new major version 2.0 for Drupal 7.

Will then backport it to Drupal 6.

Subscribe

Hi,
Could you make the 7.x-1.x-dev release downloadable. Even if it is not ready we can atleast start testing and help. Also that would mean we get a 7.x option in the version drop-down.
Thanks.
Jonathan

[edit: We can then have a 'yes' of some description in the D7 column of the comparison page http://groups.drupal.org/node/51088 just to show we are not falling behind!]

subscribing

Is the patch in #9 supposed to be applied to the dev D6 release to create a D7 version? I don't have a CVS account so I'm not exactly sure what you mean.

Could we get a that version downloadable on the project page as a 7.x-1.0-dev zip file? That would make it alot easier for people to test it/ debug it and help to progress towards a D7 release.

Jonathan

subscribe - need this

subscribe

subscribe

Subscribing.

subscribe

subscribe

Subscribing

Subscribing.

Sub!...

Would be nice to have ^^

Any status updates on this??

Whats the status on this?

same

+1 http://drupal.org/project/purr_messages has a D7 version, but seems a lot different with this.

Subscribing.

A downloadable dev version yet?
Subscribing

Fixing title so it can be distinguished from everything else in my issues queue

Subscribing

subscribing

Subscribing

Don't you guys know there is a new subscription system? Check: http://drupal.org/node/1306444

I wanted this module in my project I tried #9 but that didn't work then I have to convert 6.x-1.x-dev version for D 7. Attached code working for me, I hope this also work for you.

Thanks

Awesome! The D7 dev version should probably live on the module's download area now.

Upon install:

Notice: Undefined index: messages in better_messages_preprocess_page() (line 113 of C:\xampplite\htdocs\drupal\modules\better_messages\better_messages.module).

Upon saving the Configuration page options:

    Warning: array_merge() [function.array-merge]: Argument #1 is not an array in _form_set_class() (line 3993 of C:\xampplite\htdocs\drupal\includes\form.inc).
    Warning: array_merge() [function.array-merge]: Argument #1 is not an array in _form_set_class() (line 3993 of C:\xampplite\htdocs\drupal\includes\form.inc).
    Warning: array_merge() [function.array-merge]: Argument #1 is not an array in _form_set_class() (line 3993 of C:\xampplite\htdocs\drupal\includes\form.inc).
    Warning: array_merge() [function.array-merge]: Argument #1 is not an array in _form_set_class() (line 3993 of C:\xampplite\htdocs\drupal\includes\form.inc).
    Warning: array_merge() [function.array-merge]: Argument #1 is not an array in _form_set_class() (line 3993 of C:\xampplite\htdocs\drupal\includes\form.inc).
    Warning: implode() [function.implode]: Invalid arguments passed in theme_radios() (line 2728 of C:\xampplite\htdocs\drupal\includes\form.inc).
    Notice: Undefined index: resizable in better_messages_admin_submit() (line 209 of C:\xampplite\htdocs\drupal\modules\better_messages\better_messages.inc).
    Notice: Undefined index: messages in better_messages_preprocess_page() (line 113 of C:\xampplite\htdocs\drupal\modules\better_messages\better_messages.module).

Autofade configuration option does not currently implement.

The Undefined Index error recurs on pages where Better Messages is enabled.

change line 113 form

if ($vars['messages']) {

to

if (isset($vars['messages'])) {

and the notice will dissapear

Confirmed.

The only notice remaining shows upon Configuration save.

Notice: Undefined index: resizable in better_messages_admin_submit() (line 209 of C:\xampplite\htdocs\drupal\modules\better_messages\better_messages.inc).

I've done a fresh Drupal install, and included reszli's edit.

The array_merge(), implode(), and line 209 errors are still with us.

Hi Limbo,

Thanks for testing the code, now I have created error free patch attached. Let me know if any issue you face.

Thanks
Kuldev

Thanks Kuldev,

Your patch works great for me!
It looks like we have a serious candidate for better messages d7!

Any chance for a most recent patched D7 version?

I tried to apply the #39 patch to the code I DL from #34 and it failed.

Download 6.x-1.x-dev version of better message and patch it with #39, that should work.

thanks
Kuldev

6.x-1.x-dev + #39 = gold

Subscribing, can this be made into a released alpha version for d7 ?

Here is the updated full-fledged d7 version of better messages module. No need to apply patches anymore, download it & display awesome looking, colorful messages.

Cheers,
harsh

Getting: "better_messages_0.zip contains a version of Better Messages that is not compatible with Drupal 7.x."

Sorry Starminder, i incorrectly uploaded the 6.x version becoz of the same name. Attached is the correct one.

Cheers,
harsh

Installed your latest version, harsh.

Getting:

"Trying to get property of non-object in better_messages_preprocess() (line 115 of sites/all/modules/better_messages/better_messages.module)."

A temporary solution would be to prepend '@' symbol before the variable like if(@$variables['user']->session) { .In the mean while, i'll try & find an accurate solution.

harsh

I am using D7 where I have configured the better messages to appear in center on our website. But when we are connecting our website through mobile, better message pop up appears on top left side on the mobile site.

Please suggest what can be done in this case..

Best regards,
Sucheta.

@sucheta
This should be just some css. test you site with some mobile demo software.

I am using the version of tush@r and it works like a charm, thanks to community!

Yeah, can we please have a 7.x branch with the code from #39 for starters? We'll have that as a base and file new issues against it for any problems that might come up.

Tried using #47 and I get Notice: Undefined property: stdClass::$session in better_messages_preprocess() (line 115 of /home/[site]/public_html/sites/all/modules/better_messages/better_messages.module).

Guess I'm following this now - looking forward to using this in D7... thanks!

Just want to thank tush@r and also mention that it works perfectly in my drupa7 version.

Jaya

Sorry, spoke too soon..also getting this error:

Notice: Undefined index: original_messages in better_messages_preprocess() (line 123 of /home/content/20/8933820/html/sites/all/modules/better_messages/better_messages.module).

Hi bcobin & Jayalakshmi,

Make the change specified in comment #49. You will not get these notice errors.

Getting the same error as #56 in the logs and I have made the change suggested in #49 - any ideas? Thanks!

I think it's high time we created a 7.x branch and a 7.x-dev. Then, all problems reported here can be filed as separate issues. If we keep reporting every issue here, we'll have havoc.

Yes, i agree, it would be more convenient if there is a 7x branch

New 7.x-1.x branch as now available as development snapshot. Thanks to @devtherock.

thanks!

Thanx ;)