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.Hey there,
I know I know, 7x is about to come out and 5x is about to be deemed old school, but I was curious if there is anyway this module might be able to be ported back to 5x. We have a client project that could benefit greatly from this module. If there is no possibility or interest in this being checked out, we may go about trying to port it back (granted its technically possible) if that is OK.
Thanks,
Kevin
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | blocks404.module.v2.patch | 627 bytes | mikebell_ |
| #5 | blocks404.active.inc_.patch | 952 bytes | mikebell_ |
| #5 | blocks404.module.patch | 568 bytes | mikebell_ |
Comments
Comment #1
JohnAlbinUnfortunately, 404 Blocks uses D6's preprocess functions which allow modules to alter the variables in template files. D5 did not have any mechanism for allowing modules to alter template variables. :-(
So, you would be required to modify your theme in order to get the proper modifications. Page Title 5.x and Advanced Forum are examples of modules that require you to modify your theme for functionality. I've made some rough installation instructions in the README file.
I've spent about 45 minutes doing a quick backport. But its completely untested. So please test it. http://drupal.org/node/570706
Comment #2
vishun CreditAttribution: vishun commentedJohnAlbin, ah yeah i had a feeling there was a decent reason that it wasnt available. theme modification for 5x is totally fine, i understand that for some it may be unacceptable but if thats the case for those handful of people, then they dont really need it that bad to begin with ;)
thank you -so- much. It is much appreciated, and we will make sure to test it and give any feedback or modification suggestions based on our findings.
Comment #3
JohnAlbinWell, I guess this wasn't needed then. :-\
Comment #5
mikebell_ CreditAttribution: mikebell_ commentedThanks for the initial port, I took a look at it today and created the two attached patches to make it functional.
This is very rough but works, tested using drupal 5.20 and Zen theme.
module_load_include is not available for 5 so replaced by require_once, this probably needs additional checking around for sanity sake.
I also iterated through any regions defined in the theme to make sure that all regions are being built up. I noticed that in $vars left and right sidebar are prepended with sidebar_ not sure if this is a Zen theme thing or core.
These patches are provided as is. I'll do more testing on our dev site tomorrow morning and add any additional patches for changes I make. Feedback would be great.
Comment #6
mikebell_ CreditAttribution: mikebell_ commentedI checked a default install of Drupal 5 and the sidebar_ is there as well, odd but I guess my fix will work.
I've attached a new patch for the module file which includes some of the checking from module_load_include taken from the Drupal 6 api.
blocks404_reset_body_classes() doesn't seem to be working correctly so I'll look into that next.