Merge "[TASK] Harmonize coding style of if constructs in sysext "linkvalidator""
authorChristopher <chrissitopher@gmx.de>
Thu, 7 Jun 2012 19:19:21 +0000 (21:19 +0200)
committerGerrit Code Review <gerrit@review.typo3.org>
Thu, 7 Jun 2012 19:19:21 +0000 (21:19 +0200)
typo3/sysext/linkvalidator/classes/linktype/class.tx_linkvalidator_linktype_interface.php
typo3/sysext/linkvalidator/ext_emconf.php
typo3/sysext/linkvalidator/modfuncreport/class.tx_linkvalidator_modfuncreport.php

index 9747dab..8336f14 100644 (file)
@@ -66,6 +66,14 @@ interface tx_linkvalidator_linktype_Interface {
         */
        public function getBrokenUrl($row);
 
+       /**
+        * Generate the localized error message from the error params saved from the parsing
+        *
+        * @param array $errorParams All parameters needed for the rendering of the error message
+        * @return string Validation error message
+        */
+       public function getErrorMessage($errorParams);
+
 }
 
 ?>
\ No newline at end of file
index c0f6b72..8e59cac 100644 (file)
@@ -1,14 +1,14 @@
 <?php
 
-########################################################################
-# Extension Manager/Repository config file for ext "linkvalidator".
-#
-# Auto generated 23-10-2011 17:09
-#
-# Manual updates:
-# Only the data in the array - everything else is removed by next
-# writing. "version" and "dependencies" must not be touched!
-########################################################################
+/***************************************************************
+* Extension Manager/Repository config file for ext "linkvalidator".
+*
+* Auto generated 23-10-2011 17:09
+*
+* Manual updates:
+* Only the data in the array - everything else is removed by next
+* writing. "version" and "dependencies" must not be touched!
+***************************************************************/
 
 $EM_CONF[$_EXTKEY] = array(
        'title' => 'Linkvalidator',
index 5167c17..dace85c 100644 (file)
@@ -134,6 +134,15 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
        /** @var t3lib_pageRenderer $pageRenderer */
        protected $pageRenderer;
 
+       /** @var string $resPath Path to "linkvalidator/res/" to be used in pageRenderer */
+       protected $resPath = '';
+
+       /** @var tx_linkvalidator_linktype_Interface[] $hookObjectsArr */
+       protected $hookObjectsArr = array();
+
+       /** @var string $checkAllHtml */
+       protected $checkAllHtml = '';
+
        /**
         * Main method of modfuncreport
         *
@@ -301,7 +310,7 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
                $this->doc->setModuleTemplate(t3lib_extMgm::extPath('linkvalidator') . 'modfuncreport/mod_template.html');
 
                $this->relativePath = t3lib_extMgm::extRelPath('linkvalidator');
-               $this->pageRecord = t3lib_BEfunc::readPageAccess($this->pObj->id, $this->perms_clause);
+               $this->pageRecord = t3lib_BEfunc::readPageAccess($this->pObj->id, $GLOBALS['BE_USER']->getPagePermsClause(1));
 
                $this->isAccessibleForCurrentUser = FALSE;
                if ($this->pObj->id && is_array($this->pageRecord) || !$this->pObj->id && $this->isCurrentUserAdmin()) {
@@ -372,6 +381,7 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
                        $this->content = $this->renderBrokenLinksTable();
                } else {
                                // If no access or if ID == zero
+                               /** @var t3lib_FlashMessage $message */
                        $message = t3lib_div::makeInstance(
                                't3lib_FlashMessage',
                                $GLOBALS['LANG']->getLL('no.access'),
@@ -430,7 +440,7 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
        /**
         * Displays the table of broken links or a note if there were no broken links
         *
-        * @return html Content of the table or of the note
+        * @return string Content of the table or of the note
         */
        protected function renderBrokenLinksTable() {
                $items = $brokenLinksMarker = array();
@@ -491,8 +501,15 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
                return $content;
        }
 
-       protected function getNoBrokenLinkMessage($brokenLinksMarker) {
+       /**
+        * Replace $brokenLinksMarker['NO_BROKEN_LINKS] with localized flashmessage
+        *
+        * @param array $brokenLinksMarker
+        * @return array $brokenLinksMarker['NO_BROKEN_LINKS] replaced with flashmessage
+        */
+       protected function getNoBrokenLinkMessage(array $brokenLinksMarker) {
                $brokenLinksMarker['LIST_HEADER'] = $this->doc->sectionHeader($GLOBALS['LANG']->getLL('list.header'));
+                       /** @var t3lib_FlashMessage $message */
                $message = t3lib_div::makeInstance(
                        't3lib_FlashMessage',
                        $GLOBALS['LANG']->getLL('list.no.broken.links'),
@@ -544,11 +561,10 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
         */
        protected function renderTableRow($table, array $row, $brokenLinksItemTemplate) {
                $markerArray = array();
-               if (is_array($row) && !empty($row['link_type'])) {
-                       if (($hookObj = $this->hookObjectsArr[$row['link_type']])) {
-                               $brokenUrl = $hookObj->getBrokenUrl($row);
-                       }
-               }
+               $fieldName = '';
+                       // Restore the linktype object
+               $hookObj = $this->hookObjectsArr[$row['link_type']];
+               $brokenUrl = $hookObj->getBrokenUrl($row);
 
                $params = '&edit[' . $table . '][' . $row['record_uid'] . ']=edit';
                $actionLinks = '<a href="#" onclick="' .
@@ -575,7 +591,7 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
                        }
                }
                        // Fallback, if there is no label
-               $fieldName = $fieldName ? $fieldName : $row['field'];
+               $fieldName = !empty($fieldName) ? $fieldName : $row['field'];
 
                        // column "Element"
                $element = t3lib_iconWorks::getSpriteIconForRecord($table, $row, array('title' => $table . ':' . $row['record_uid']));
@@ -765,4 +781,4 @@ class tx_linkvalidator_ModFuncReport extends t3lib_extobjbase {
        }
 }
 
-?>
\ No newline at end of file
+?>