EXTMVC:
authorJochen Rau <j.rau@web.de>
Tue, 24 Mar 2009 23:22:13 +0000 (23:22 +0000)
committerJochen Rau <j.rau@web.de>
Tue, 24 Mar 2009 23:22:13 +0000 (23:22 +0000)
* Removed obsolete comments

typo3/sysext/extbase/Classes/Persistence/Exception/TooDirty.php
typo3/sysext/extbase/Classes/Persistence/Mapper/ObjectRelationalMapper.php
typo3/sysext/extbase/Classes/Persistence/ObjectStorage.php
typo3/sysext/extbase/Classes/Persistence/Session.php
typo3/sysext/extbase/Classes/Request.php
typo3/sysext/extbase/Classes/View/Helper/TranslateHelper.php
typo3/sysext/extbase/Classes/View/TemplateView.php
typo3/sysext/extbase/Classes/Web/Response.php

index 8ba51b3..60c7963 100644 (file)
@@ -29,8 +29,6 @@
  * @subpackage extmvc
  * @version $ID:$
  */
-// SK: What does "TooDirty" mean? The UID has been modified? I'd suggest a more understandable name :-)
-// JR: That's a class name Karsten has chosen. In deed, the exception will be thrown if UID has been modified 
 class TooDirty extends TX_EXTMVC_Persistence_Exception {
 }
 
index 947c13f..225479d 100644 (file)
@@ -172,7 +172,6 @@ class TX_EXTMVC_Persistence_Mapper_ObjectRelationalMapper implements t3lib_Singl
         */
        // SK: I Need to check this method more thoroughly.
        // SK: Are loops detected during reconstitution?
-       // SK: What about "1:1" relations?
        protected function reconstituteObjects($dataMap, array $rows) {
                $objects = array();             
                foreach ($rows as $row) {
index dadde8e..06eaa33 100644 (file)
@@ -31,8 +31,6 @@
  * @version $ID:$
  */
 class TX_EXTMVC_Persistence_ObjectStorage implements Iterator, Countable, ArrayAccess {
-// SK: Why not use SplObjectStorage?
-// JR: SplObjectStorage isn't fully implemented in PHP 5.2.x
 
        /**
         * The array holding references of the stored objects
index f2c0d60..a8347fa 100644 (file)
@@ -93,8 +93,6 @@ class TX_EXTMVC_Persistence_Session implements t3lib_singleton {
         * @param TX_EXTMVC_DomainObject_AbstractDomainObject $object
         * @return void
         */
-       // SK: When is this method called?
-       // JR: $this->unregisterObject() to clear the persistence session
        public function unregisterAddedObject(TX_EXTMVC_DomainObject_AbstractDomainObject $object) {
                $this->addedObjects->detach($object);
        }
index ff69c50..cdf9c69 100644 (file)
@@ -293,27 +293,6 @@ class TX_EXTMVC_Request {
        }
 
        /**
-        * Sets the requested representation format
-        *
-        * @param string $format The desired format, something like "html", "xml", "png", "json" or the like.
-        * @return void
-        */
-       // SK: To be discussed. I think "*format" can be removed, or do we support multiple request formats currently??
-       public function setFormat($format) {
-               if (!preg_match(self::PATTERN_MATCH_FORMAT, $format)) throw new TX_EXTMVC_Exception_InvalidFormat('An invalid request format (' . $format . ') was given.', 1218015038);
-               $this->format = $format;
-       }
-
-       /**
-        * Returns the requested representation format
-        *
-        * @return string The desired format, something like "html", "xml", "png", "json" or the like.
-        */
-       public function getFormat() {
-               return $this->format;
-       }
-
-       /**
         * Returns the value of the specified argument
         *
         * @param string $argumentName Name of the argument
index 1d57f4f..4f6c0ac 100644 (file)
@@ -29,7 +29,6 @@
  * @subpackage extmvc
  * @version $ID:$
  */
-// TODO Enable caching for translations?
 class TX_EXTMVC_View_Helper_TranslateHelper extends TX_EXTMVC_View_Helper_AbstractHelper {
 
        /**
@@ -90,14 +89,12 @@ class TX_EXTMVC_View_Helper_TranslateHelper extends TX_EXTMVC_View_Helper_Abstra
                        }
                }
 
-               // Read the strings in the required charset (since TYPO3 4.2)
                $this->LOCAL_LANG = t3lib_div::readLLfile($languageFilePath, $this->languageKey, $GLOBALS['TSFE']->renderCharset);
                if ($this->alternativeLanguageKey)      {
                        $tempLOCAL_LANG = t3lib_div::readLLfile($languageFilePath, $this->alternativeLanguageKey);
                        $this->LOCAL_LANG = array_merge(is_array($this->LOCAL_LANG) ? $this->LOCAL_LANG : array(), $tempLOCAL_LANG);
                }
 
-               // TODO Overlaying labels from TypoScript (including fictitious language keys for non-system languages!):
                $configurationManager = t3lib_div::makeInstance('TX_EXTMVC_Configuration_Manager');
                $settings = $configurationManager->getSettings($this->request->getControllerExtensionKey());
                if (is_array($settings['_LOCAL_LANG'])) {
index 879fa20..a1285e8 100644 (file)
@@ -34,8 +34,6 @@ require_once(PATH_t3lib . 'class.t3lib_parsehtml.php');
  * @version $ID:$
  * @scope prototype
  */
-// TODO Should we provide IF THEN ELSE statements?
-// TODO Should we allow chaining (e.g. pervorm a convertion after translation)?
 class TX_EXTMVC_View_TemplateView extends TX_EXTMVC_View_AbstractView {
 
        /**
@@ -165,7 +163,7 @@ class TX_EXTMVC_View_TemplateView extends TX_EXTMVC_View_AbstractView {
         */
        protected function loadTemplateFile($templateFilePath) {
                $templateSource = file_get_contents(t3lib_extMgm::extPath(strtolower($this->request->getControllerExtensionKey())) . $templateFilePath, FILE_TEXT);
-               if (!$templateSource) throw new RuntimeException('The template file "' . $templateFilePath . '" was not found.', 1225709595); // TODO Specific exception
+               if (!$templateSource) throw new RuntimeException('The template file "' . $templateFilePath . '" was not found.', 1225709595);
                return $templateSource;
        }
 
@@ -181,7 +179,6 @@ class TX_EXTMVC_View_TemplateView extends TX_EXTMVC_View_AbstractView {
                } else {
                        $templateSource = $this->templateSource;
                }
-               // TODO Throw exception if a template was not defined
                $content = $this->renderTemplate($templateSource, $this->contextVariables);
                $this->removeUnfilledMarkers($content);
                return $content;
@@ -256,12 +253,10 @@ class TX_EXTMVC_View_TemplateView extends TX_EXTMVC_View_AbstractView {
                                        $content = $object->$possibleMethodName(); // Properties should be already secure (XSS)
                                }
                        } elseif (!empty($variables[$contextVariableName])) {
-                               // TODO Maybe filter_var() is too much. Make it configurable? Or should we check it at assign time?
                                $content = filter_var($variables[$contextVariableName], FILTER_SANITIZE_STRING);
                        }
 
                        if (!empty($viewHelperName)) {
-                               // TODO Make this configurable by injecting the viewHelpers?
                                $viewHelperClassName = 'TX_EXTMVC_View_Helper_' . $viewHelperName . 'Helper';
                                $viewHelper = $this->getViewHelper($viewHelperClassName);
                                $content = $viewHelper->render($this, $arguments, $templateSource, $variables);
index e8f2ff9..43610b2 100755 (executable)
@@ -30,7 +30,6 @@
  * @version $ID:$
  * @scope prototype
  */
-// SK: Please implement setHeader(), getHeaders() and sendHeaders() as it is in FLOW3 to send custom HTTP headers.
 class TX_EXTMVC_Web_Response extends TX_EXTMVC_Response {
 
        /**