I'm working on a D7 port of this module. Patch coming

Comments

Status:Active» Patch (to be ported)
StatusFileSize
new17.18 KB

Patch to go from 6.x-1.x-dev to a Drupal 7 version attached

Status:Patch (to be ported)» Needs work
StatusFileSize
new17.21 KB

+++ b/admin_message.jsundefined
@@ -1,13 +1,11 @@
+    $("#block-admin-message-admin-message a.admin-message-close").click(function() {

give the 'event' parameter to the function

+++ b/admin_message.jsundefined
@@ -1,13 +1,11 @@
       return false;

event.preventDeafault() rather than return false

fixed in attached patch

StatusFileSize
new17.1 KB

drupal_add_js and drupal_add_css api changes...

sandbox for D7 created at http://drupal.org/sandbox/jelles/1254090

Jelle_S: Are you still interested in working on a Drupal 7 port? If you update your code for the latest changes in the 6.x-1.x branch, I'd be happy to make a 7.x-1.x branch here and put your code in it. I'd do it now, but I'm worried about losing the functionality that was just merged.

Is any work being done on the Drupal 7 port at this point, or has the port been dropped?

The patch that Jelle_S posted will still work against a 6.x-1.x-dev version from when the patch was written. However, that is missing many of the most current features of this module. As far as I know, no one is working on a Drupal 7 port. If you're interested in continuing Jelle_S's working and updating the patch for the latest features, I'd be happy to review and accept it.

I have admin_message.module working on a D7 website. I patched it to work with translation.module on a multi-language website (the same message in multiple languages, closing a message in one language closes the corresponding messages as well).

I'd like to provide the patches, but I cannot (black box) reverse engineer the changes from Jelle_S's patch against the current D6 feature set.

I'm afraid if you (dsnopek) are not interested in a D7 port, it will not be possible for another single person else to do the work and provide a patch.

A first step might be to open a 7.x branch and apply the patch provided by Jelle_S. And dsnopek delivers a list of features that should be ported to D7.

Through the magic of "git rebase" I've created a patch that combines the recent changes to 6.x-1.x and the changes in Jelle_S's sandbox. :-)

I also reviewed the patch a little bit and there were a number of things in it that I didn't think were quite right. So, I've also created an updated patch which I hope is better.

However, I haven't tested either of them!

@schoobidoo, Could you try this patch and see if it works? Once we can settle on a patch for D7, I'll create the 7.x-1.x branch and you can submit your multilingual patch.

FYI all the changes that were made since Jelle_S's last patch are in the Git log, which lists the specific issues in the issue queue:

So, for the record, I don't agree with this statement:

I'm afraid if you (dsnopek) are not interested in a D7 port, it will not be possible for another single person else to do the work and provide a patch.

Anyone who can operate git could have combined the changes or looked up which functionality I'm worried about losing. The real work is testing and finishing the port, which is something I'm not personally interested in working on at this time.

However, I will happily review and accept working patches!

Thanks for your work on this dsnopek. Would you mind releasing a 7.x-dev version of the module? Everyone should realize that a dev version is not to be used on a production site, but it seems like you might get more people testing out the module that way...

@JordanMagnuson: As soon as we settle on this patch, I will, definetyl! Right now, the latest patch is completely untested. So, either someone here please test it and let me know or I'll get around to it eventually. :-)

Issue summary:View changes

Updated issue summary.