Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I'm working on a D7 port of this module. Patch coming
Comment | File | Size | Author |
---|---|---|---|
#9 | admin_message-d7-1247462-jelles-merge.do-not-use.patch | 16.26 KB | dsnopek |
#9 | admin_message-d7-1247462-9.patch | 15.87 KB | dsnopek |
#3 | 1247462-3.patch | 17.1 KB | Jelle_S |
#2 | 1247462-2.patch | 17.21 KB | Jelle_S |
#1 | 1247462.patch | 17.18 KB | Jelle_S |
Comments
Comment #1
Jelle_SPatch to go from 6.x-1.x-dev to a Drupal 7 version attached
Comment #2
Jelle_Sgive the 'event' parameter to the function
event.preventDeafault() rather than return false
fixed in attached patch
Comment #3
Jelle_Sdrupal_add_js and drupal_add_css api changes...
Comment #4
Jelle_Ssandbox for D7 created at http://drupal.org/sandbox/jelles/1254090
Comment #5
dsnopekJelle_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.
Comment #6
JordanMagnuson CreditAttribution: JordanMagnuson commentedIs any work being done on the Drupal 7 port at this point, or has the port been dropped?
Comment #7
dsnopekThe 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.
Comment #8
Anonymous (not verified) CreditAttribution: Anonymous commentedI 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.
Comment #9
dsnopekThrough 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:
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!
Comment #10
JordanMagnuson CreditAttribution: JordanMagnuson commentedThanks 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...
Comment #11
dsnopek@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. :-)
Comment #11.0
dsnopekUpdated issue summary.
Comment #12
igorik CreditAttribution: igorik commentedBump!
Is there a chance to get the d7 version (at least patched file) for download?
thanks
Igor