[FEATURE] RTE: Cleanup in SpellCheckingController 41/33241/2
authorStanislas Rolland <typo3@sjbr.ca>
Fri, 10 Oct 2014 02:14:17 +0000 (22:14 -0400)
committerStanislas Rolland <typo3@sjbr.ca>
Fri, 10 Oct 2014 02:15:58 +0000 (04:15 +0200)
Add comments. Clearer Aspell command construction.

Releases: master
Resolves: #62138
Change-Id: Ib553d782bc2602c53540e7a532f14cdd4e62fbb8
Reviewed-on: http://review.typo3.org/33241
Reviewed-by: Stanislas Rolland <typo3@sjbr.ca>
Tested-by: Stanislas Rolland <typo3@sjbr.ca>
typo3/sysext/rtehtmlarea/Classes/Controller/SpellCheckingController.php

index 8f4892c..13073ec 100644 (file)
@@ -472,7 +472,7 @@ var selectedDictionary = "' . $this->dictionary . '";
        }
 
        /**
-        * @todo Define visibility
+        * Handler for the opening of a tag
         */
        public function startHandler($xml_parser, $tag, $attributes) {
                if (strlen($this->xmlCharacterData)) {
@@ -513,7 +513,7 @@ var selectedDictionary = "' . $this->dictionary . '";
        }
 
        /**
-        * @todo Define visibility
+        * Handler for the closing of a tag
         */
        public function endHandler($xml_parser, $tag) {
                if (strlen($this->xmlCharacterData)) {
@@ -549,7 +549,7 @@ var selectedDictionary = "' . $this->dictionary . '";
        }
 
        /**
-        * @todo Define visibility
+        * Handler for the content of a tag
         */
        public function spellCheckHandler($xml_parser, $string) {
                $incurrent = array();
@@ -590,8 +590,16 @@ var selectedDictionary = "' . $this->dictionary . '";
                                        if (!fclose($filehandle)) {
                                                echo 'SpellChecker tempfile close error';
                                        }
-                                       $catCommand = TYPO3_OS == 'WIN' ? 'type' : 'cat';
-                                       $AspellCommand = $catCommand . ' ' . escapeshellarg($tmpFileName) . ' | ' . $this->AspellDirectory . ' -a check --mode=none --sug-mode=' . escapeshellarg($this->pspellMode) . ($this->personalDictionaryPath ? ' --home-dir=' . escapeshellarg($this->personalDictionaryPath) : '') . ' --lang=' . escapeshellarg($this->dictionary) . ' --encoding=' . escapeshellarg($this->aspellEncoding) . ' 2>&1';
+                                       $catCommand = TYPO3_OS === 'WIN' ? 'type' : 'cat';
+                                       $AspellCommand = $catCommand . ' ' . escapeshellarg($tmpFileName) . ' | '
+                                               . $this->AspellDirectory
+                                               . ' -a check'
+                                               . ' --mode=none'
+                                               . ' --sug-mode=' . escapeshellarg($this->pspellMode)
+                                               . ($this->personalDictionaryPath ? ' --home-dir=' . escapeshellarg($this->personalDictionaryPath) : '')
+                                               . ' --lang=' . escapeshellarg($this->dictionary)
+                                               . ' --encoding=' . escapeshellarg($this->aspellEncoding)
+                                               . ' 2>&1';
                                        $AspellAnswer = shell_exec($AspellCommand);
                                        $AspellResultLines = array();
                                        $AspellResultLines = GeneralUtility::trimExplode(LF, $AspellAnswer, TRUE);
@@ -634,14 +642,14 @@ var selectedDictionary = "' . $this->dictionary . '";
        }
 
        /**
-        * @todo Define visibility
+        * Handler for collecting data within a tag
         */
        public function collectDataHandler($xml_parser, $string) {
                $this->xmlCharacterData .= $string;
        }
 
        /**
-        * @todo Define visibility
+        * Default handler for the xml parser
         */
        public function defaultHandler($xml_parser, $string) {
                $this->text .= $string;