CVS edit link for Gabriel Radic

I have a small module called Body Wrapper that would be useful to others. I've previously simply posted it in the forum, but it makes more sense to give it a home.

Also, my company often improve or adapt existing modules and themes and would like to contribute back my changes. Some of the modules we work on are Node Hierarchy, Ubercart and other.

CommentFileSizeAuthor
#1 bodywrapper.zip6.53 KBGabriel R.

Comments

Gabriel R.’s picture

Gabriel R. commented
Component: Miscellaneous » Code
Assigned: Unassigned » Gabriel R.
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new bodywrapper.zip6.53 KB

An initial, stable and recommended version for Drupal 6.x

gerhard killesreiter’s picture

gerhard killesreiter commented

I think a 8 line module is a bit pushing the definition of "module".

This is also something which probably should be done in the theming layer.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Component: Code » Miscellaneous
Assigned: Gabriel R. » Unassigned

I agree with Gerhard; 8 lines of code doesn't make a module, and the code should be as Gerhard suggested.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Closed (won't fix)

The proposed module is just 8 lines of code. I am setting this report as "won't fix".

Bill_El’s picture

Bill_El commented
Project: Drupal.org CVS applications » Drupal core
Version: » 6.x-dev
Component: Miscellaneous » theme system
Category: task » feature
Status: Closed (won't fix) » Active

Just wondering how could this code be implemented into the template.php file?

Seems like a better place for it.

Thanks.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Project: Drupal core » Drupal.org CVS applications
Version: 6.x-dev »
Component: theme system » Miscellaneous
Category: feature » task
Status: Active » Closed (won't fix)

Please leave the issue in the Drupal.org CVS applications queue where they are.

If you want to open a feature request, then open a new report; keep in mind that feature requests for Drupal 6 are probably not implemented.