Anonymous | Login | Signup for a new account | 2023-03-21 02:21 CDT | ![]() |
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories |
Viewing Issue Simple Details [ Jump to Notes ] [ Wiki ] | [ View Advanced ] [ Issue History ] [ Print ] | ||||||
ID | Category | Type | Reproducibility | Date Submitted | Last Update | ||
0001007 | [In-Portal CMS] Optimization | refactoring | always | 2011-02-21 15:09 | 2011-03-30 09:58 | ||
Reporter | Dmitry | View Status | public | Project Name | Custom (Dev. Kit) | ||
Assigned To | Dmitry | Developer | |||||
Priority | normal | Resolution | fixed | Fixed in Version | 1.1.2-RC1 | ||
Status | closed | Product Version | 1.1.2-B1 | Target Version | 1.1.2 | ||
Time Estimate | No estimate | ||||||
Summary | 0001007: Minimize number of Extended Units in Sections folder | ||||||
Description |
We have recently came across the issue when in some projects we have too many PHP files in a single Units/Sections folder of Custom module. As you might know this folder contains Extended classes which are used to override default functionality of In-Portal. As a result it's getting very hard to quickly locate necessary files once you reach 20+ files in there, plus they all start with e_OriginalFileName.php At this point, we should throughly discussion all options of making this simpler to work with. I ask everyone (including Intechnic employees) to participate in this critical decision making process. Solution: No CORE changes required here! Under Units/Sections we create a sub-folders for each corresponding Unit name (that are extending) and put replacement files inside. Example: - it was - /modules/custom/units/sections/e_order_eh.php - it will be - /modules/custom/units/sections/orders/e_order_eh.php Do all Extended Units for Custom module like this. |
||||||
Additional Information | |||||||
Tags | No tags attached. | ||||||
Reference | https://groups.google.com/d/topic/in-portal-dev/kEuR0sp95m4/discussion | ||||||
Change Log Message | Minimize number of extended Units in "sections" folder | ||||||
Estimate Points | 0 | ||||||
Attached Files | |||||||
|
![]() |
|
![]() |
|
![]() Dmitry (manager) 2011-02-28 13:32 |
Fix committed to 1.1.x branch. Commit Message: Fixes 0001007: Minimize number of Extended Units in Sections folder |
![]() Dmitry (manager) 2011-03-30 09:58 |
Closing, since 1.1.2 release has been released. |
![]() |
|||
Modules :: Custom: 1.1.x r14239 Timestamp: 2011-02-28 13:32:18 Author: Dmitry [ Details ] [ Diff ] |
Fixes 0001007: Minimize number of Extended Units in Sections folder | ||
add - /modules/custom/branches/1.1.x/units/sections/articles | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/articles/e_article_eh.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/articles/e_article_tp.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/categories | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/categories/e_category_eh.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/categories/e_category_tp.php | [ File ] | ||
mod - /modules/custom/branches/1.1.x/units/sections/custom_eh.php | [ Diff ] [ File ] | ||
rm - /modules/custom/branches/1.1.x/units/sections/e_article_eh.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_article_tp.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_category_eh.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_category_tp.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_link_eh.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_link_tp.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_product_eh.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_product_tp.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_review_eh.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_user_eh.php | |||
rm - /modules/custom/branches/1.1.x/units/sections/e_user_tp.php | |||
add - /modules/custom/branches/1.1.x/units/sections/links | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/links/e_link_eh.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/links/e_link_tp.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/products | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/products/e_product_eh.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/products/e_product_tp.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/reviews | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/reviews/e_review_eh.php | [ File ] | ||
mod - /modules/custom/branches/1.1.x/units/sections/sections_config.php | [ Diff ] [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/users | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/users/e_user_eh.php | [ File ] | ||
add - /modules/custom/branches/1.1.x/units/sections/users/e_user_tp.php | [ File ] | ||
![]() |
|||
Date Modified | Username | Field | Change |
2011-03-30 09:58 | Dmitry | Note Added: 0003379 | |
2011-03-30 09:58 | Dmitry | Status | resolved => closed |
2011-02-28 13:32 | Dmitry | Changeset attached | 1.1.x r14239 |
2011-02-28 13:32 | Dmitry | Note Added: 0003282 | |
2011-02-28 13:32 | Dmitry | Status | active => resolved |
2011-02-28 13:32 | Dmitry | Fixed in Version | => 1.1.2-RC1 |
2011-02-28 13:32 | Dmitry | Resolution | open => fixed |
2011-02-28 13:32 | Dmitry | Assigned To | => Dmitry |
2011-02-21 15:11 | Dmitry | Type | bug report => refactoring |
2011-02-21 15:09 | Dmitry | Description Updated | View Revisions |
2011-02-21 15:09 | Dmitry | New Issue | |
2011-02-21 15:09 | Dmitry | Reference | => https://groups.google.com/d/topic/in-portal-dev/kEuR0sp95m4/discussion |
2011-02-21 15:09 | Dmitry | Change Log Message | => Minimize number of extended Units in "sections" folder |
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories |
Web Development by Intechnic![]() In-Portal Open Source CMS |