Markers
Table of Contents
| Horde/Itip/Event/Vevent.php | 2 |
| Horde/Itip/Event/Vtodo.php | 3 |
| Horde/Itip/Resource/Identity.php | 1 |
| Horde/Itip/Response/Vtodo.php | 3 |
| Horde/Itip.php | 1 |
Vevent.php
| Type | Line | Description |
|---|---|---|
| TODO | 32 | Clean this class up. Accessing private methods for copying the object is not nice. Reconsider if an interface is really needed. See also PMD report. |
| TODO | 120 | Parse mailto using parse_url |
Vtodo.php
| Type | Line | Description |
|---|---|---|
| TODO | 34 | Clean this class up. Accessing private methods for copying the object is not nice. Reconsider if an interface is really needed. See also PMD report. |
| TODO | 61 | H6, either change name of method, or add method specific to Vtodo. |
| TODO | 122 | Parse mailto using parse_url |
Identity.php
| Type | Line | Description |
|---|---|---|
| TODO | 48 | Parse mailto using parse_url |
Vtodo.php
| Type | Line | Description |
|---|---|---|
| TODO | 41 | For H6, look at protected/private visibility of parent class' methods and properties. Needed to provide duplicated methods, like __construct due to private members. Might be able to combine classes once type hints are fixed/changed in the Horde_Itip_Event_* classes. |
| TODO | 95 | Refactor this along with parent class. This method name is confusing, but necessary due to the parent class' method name. It returns a vTodo, not vEvent. |
| TODO | 125 | I tried to abstract just the .ics filename, but due to private members in the parent class, must override the entire method here. |
Itip.php
| Type | Line | Description |
|---|---|---|
| TODO | 151 | This should be combined with self::factory. |