Index: branches/ApiEdit_Vodafone/includes/EditPage.php |
— | — | @@ -1601,7 +1601,6 @@ |
1602 | 1602 | //$baseRevision=true; |
1603 | 1603 | if( is_null( $baseRevision ) ) { |
1604 | 1604 | wfProfileOut( $fname ); |
1605 | | -//break; |
1606 | 1605 | return false; |
1607 | 1606 | } |
1608 | 1607 | $baseText = $baseRevision->getText(); |
— | — | @@ -2136,103 +2135,80 @@ |
2137 | 2136 | case self::AS_END: |
2138 | 2137 | //wfProfileOut( $fname ); |
2139 | 2138 | return true; |
2140 | | - break; |
2141 | | - |
| 2139 | + |
2142 | 2140 | case self::AS_SUCCESS_UPDATE: |
2143 | 2141 | return false; |
2144 | | - break; |
2145 | | - |
| 2142 | + |
2146 | 2143 | case self::AS_MAX_ARTICLE_SIZE_EXCEDED: |
2147 | 2144 | return true; |
2148 | | - break; |
2149 | | - |
| 2145 | + |
2150 | 2146 | case self::AS_TEXTBOX_EMPTY: |
2151 | 2147 | return true; |
2152 | | - break; |
2153 | | - |
| 2148 | + |
2154 | 2149 | case self::AS_SUMMARY_NEEDED_B: |
2155 | 2150 | return true; |
2156 | | - break; |
2157 | | - |
| 2151 | + |
2158 | 2152 | case self::AS_SUMMARY_NEEDED_A: |
2159 | 2153 | return true; |
2160 | | - break; |
2161 | | - |
| 2154 | + |
2162 | 2155 | case self::AS_CONFLICT_DETECTED: |
2163 | 2156 | return true; |
2164 | | - break; |
2165 | | - |
| 2157 | + |
2166 | 2158 | case self::AS_SUCCESS_NEW_ARTICLE: |
2167 | 2159 | return false; |
2168 | | - break; |
2169 | | - |
| 2160 | + |
2170 | 2161 | case self::AS_BLANK_ARTICLE: |
2171 | 2162 | $wgOut->redirect( $this->mTitle->getFullURL() ); |
2172 | 2163 | return false; |
2173 | | - break; |
2174 | | - |
| 2164 | + |
2175 | 2165 | case self::AS_NO_CREATE_PERMISSION; |
2176 | 2166 | $this->noCreatePermission(); |
2177 | 2167 | return; |
2178 | | - break; |
2179 | | - |
| 2168 | + |
2180 | 2169 | case self::AS_ARTICLE_WAS_DELETED: |
2181 | 2170 | return true; |
2182 | | - break; |
2183 | | - |
| 2171 | + |
2184 | 2172 | case self::AS_RATE_LIMITED: |
2185 | 2173 | $wgOut->rateLimited(); |
2186 | 2174 | return false; |
2187 | | - break; |
2188 | | - |
| 2175 | + |
2189 | 2176 | case self::AS_READ_ONLY_PAGE: |
2190 | 2177 | $wgOut->readOnlyPage(); |
2191 | 2178 | return false; |
2192 | | - break; |
2193 | | - |
| 2179 | + |
2194 | 2180 | case self::AS_READ_ONLY_PAGE_LOGGED: |
2195 | 2181 | $wgOut->readOnlyPage(); |
2196 | 2182 | return false; |
2197 | | - break; |
2198 | | - |
| 2183 | + |
2199 | 2184 | case self::AS_READ_ONLY_PAGE_ANON: |
2200 | 2185 | $this->userNotLoggedInPage(); |
2201 | 2186 | return false; |
2202 | | - break; |
2203 | | - |
| 2187 | + |
2204 | 2188 | case self::AS_CONTENT_TOO_BIG: |
2205 | 2189 | return true; |
2206 | | - break; |
2207 | | - |
| 2190 | + |
2208 | 2191 | case self::AS_BLOCKED_PAGE_FOR_USER: |
2209 | 2192 | $this->blockedPage(); |
2210 | 2193 | return false; |
2211 | | - break; |
2212 | | - |
| 2194 | + |
2213 | 2195 | case self::AS_EXIST_HOOK_ERROR_A: |
2214 | 2196 | return false; |
2215 | | - break; |
2216 | | - |
| 2197 | + |
2217 | 2198 | case self::AS_SPAM_ERROR: |
2218 | 2199 | $this->spamPage ( $matches[0] ); |
2219 | 2200 | return false; |
2220 | | - break; |
2221 | | - |
| 2201 | + |
2222 | 2202 | case self::AS_FILTERING: |
2223 | 2203 | return false; |
2224 | | - break; |
2225 | | - |
| 2204 | + |
2226 | 2205 | case self::AS_EXIST_HOOK_ERROR_B: |
2227 | 2206 | return false; |
2228 | | - break; |
2229 | | - |
| 2207 | + |
2230 | 2208 | case self::AS_EXIST_HOOK_ERROR_C: |
2231 | 2209 | return true; |
2232 | | - break; |
2233 | | - |
| 2210 | + |
2234 | 2211 | // case DEFAULT: |
2235 | 2212 | // return true; |
2236 | | -// break; |
2237 | 2213 | } |
2238 | 2214 | } |
2239 | 2215 | |