Add some todos (and use phpdoc rather than the TODO file).

diff --git a/framework/Itip/TODO b/framework/Itip/TODO
deleted file mode 100644 (file)
index 4580e73..0000000
--- a/framework/Itip/TODO
+++ /dev/null
@@ -1,3 +0,0 @@
- 
- - Merge with Horde Imp (Changes [Horde]: Extended message body; improved From, Changes [Kolab]: Do not automatically set the sequence attribute to 0 if it does not exist -> http://bugs.horde.org/ticket/4863; use lower case mailto -> http://bugs.horde.org/ticket/4872; Missing UID in the vEvent causes failure;
- - Use parse_url

diff --git a/framework/Itip/lib/Horde/Itip/Event/Vevent.php b/framework/Itip/lib/Horde/Itip/Event/Vevent.php
index 84b1e8d..ced204f 100644 (file)
--- a/framework/Itip/lib/Horde/Itip/Event/Vevent.php
+++ b/framework/Itip/lib/Horde/Itip/Event/Vevent.php
@@ -26,6 +26,10 @@
  * @author   Gunnar Wrobel <wrobel@pardus.de>
  * @license  http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html LGPL
  * @link     http://pear.horde.org/index.php?package=Itip
+ *
+ * @todo 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.
  */
 class Horde_Itip_Event_Vevent
 implements Horde_Itip_Event
@@ -112,6 +116,8 @@ implements Horde_Itip_Event
      * Return the organizer of the iTip event.
      *
      * @return string The organizer of the event.
+     *
+     * @todo Parse mailto using parse_url
      */
     public function getOrganizer()
     {

diff --git a/framework/Itip/lib/Horde/Itip/Resource/Identity.php b/framework/Itip/lib/Horde/Itip/Resource/Identity.php
index bd45a1a..b285574 100644 (file)
--- a/framework/Itip/lib/Horde/Itip/Resource/Identity.php
+++ b/framework/Itip/lib/Horde/Itip/Resource/Identity.php
@@ -51,6 +51,7 @@ implements Horde_Itip_Resource
      * @param array              $attendees The attendees of the invitation.
      * @param string             $reply_to  The selected identity for sending the
      *                                      reply.
+     * @todo Parse mailto using parse_url
      */
     public function __construct($identity, $attendees, $reply_to)
     {

diff --git a/framework/Itip/package.xml b/framework/Itip/package.xml
index c1f3430..c805b77 100644 (file)
--- a/framework/Itip/package.xml
+++ b/framework/Itip/package.xml
@@ -98,7 +98,6 @@
     </dir> <!-- /test/Horde -->
    </dir> <!-- /test -->
    <file name="COPYING" role="doc" />
-   <file name="TODO" role="doc" />
   </dir> <!-- / -->
  </contents>
  <dependencies>