Posted by ardas on November 24, 2010 at 1:14pm
2 followers
Jump to:
| Project: | Block Class |
| Version: | 6.x-1.3 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (won't fix) |
Issue Summary
Greetings,
We noticed that you put all code into .module file which is not the best practice because all code is loaded on all pages even when it is not needed. We suggest to move admin pages code into block_class.admin.inc. I have attached the new version of 1.3 to this post.
It also includes our changes from another our post regarding human readable list of classes...
| Attachment | Size |
|---|---|
| block_class_new.zip | 1.89 KB |
Comments
#1
ardas, can you roll this as a patch against the dev version?
#2
The module is only 108 lines, supply patch against dev if you want this fixed.