Impediment #29636
list view: fix inconsistencies where to look for configuration
| Status: | Closed | Start date: | 2011-09-10 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assignee: | Thomas Kowtsch | % Done: | 100% |
|
| Category: | Refactoring | Spent time: | 1.00 hour | |
| Target version: | 1.5 | |||
| Votes: | 0 |
Description
Within class.tx_cal_listview, different ways are taken to look for configuration:
- hard-coded view.list.xxx (maxEvents, showLongEventsInEachWrapper)
- dynamic $this->conf['view'](pagebrwoser settings)
My suggestion is to move everything to the dynamic way to stay consistent and flexible.
History
Updated by Thomas Kowtsch almost 2 years ago
- Target version set to 1.5
Updated by Thomas Kowtsch over 1 year ago
- Category set to Refactoring
Updated by Thomas Kowtsch over 1 year ago
- Assignee set to Thomas Kowtsch
- % Done changed from 0 to 100
Commited alignement in r59277 - used configuration is now always taken from list view.
Updated by Thomas Kowtsch about 1 year ago
- Status changed from New to Closed