Task #39977
Allow oneliners for throw statements
Status: | Rejected | Start date: | 2012-08-19 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Andy Grunwald | % Done: | 0% |
|
Category: | ControlStructures | |||
Target version: | 0.0.6 | |||
Branch: | v5 | Tags: | typo3flow |
Description
The FLOW3 CGL allows oneline if statements if a throw statement is used: http://flow3.typo3.org/documentation/codingguidelines/php.html#if-statements
For this the sniffing should be loosened a bit to allow the oneliners in those cases
History
#1 Updated by Stefano Kowalke almost 3 years ago
- Category set to ControlStructures
- Status changed from New to Needs Feedback
This topic should discussed first, because it is not part of TYPO3v4 CGL yet. It is not forbidden nor allowed like in FLOW3 CGL . Please open a report at http://forge.typo3.org/projects/typo3v4-doc_core_cgl/issues.
#2 Updated by Rens Admiraal almost 3 years ago
I don't see why this should be discussed first as I ask for adding the rules to the FLOW3 rulesset.
More important, we spoke about this rule in Copenhagen and we'll probably remove the exception from the cgl. This would mean this issue can be closed. I'll update this issue as soon as I'm sure we'll change the cgl.
#3 Updated by Rens Admiraal almost 3 years ago
I don't see why this should be discussed first as I ask for adding the rules to the FLOW3 rulesset.
More important, we spoke about this rule in Copenhagen and we'll probably remove the exception from the cgl. This would mean this issue can be closed. I'll update this issue as soon as I'm sure we'll change the cgl.
#4 Updated by Andy Grunwald almost 3 years ago
- Target version set to 0.0.5
- Branch set to v5
#5 Updated by Andy Grunwald almost 3 years ago
Okay. THX rens.
We will wait for feedback.
#6 Updated by Stefano Kowalke almost 3 years ago
- Target version changed from 0.0.5 to 0.0.6
#7 Updated by Rens Admiraal almost 3 years ago
It is indeed decided to remove this exception, so this one can be closed. We will update the CGL on our website.
#8 Updated by Andy Grunwald almost 3 years ago
- Status changed from Needs Feedback to Closed
- Assigned To set to Andy Grunwald
As Rens mentioned the affected change (https://review.typo3.org/#/c/15806/) was merged.
This ticket is not needed anymore.
#9 Updated by Stefano Kowalke over 2 years ago
- Status changed from Closed to Rejected
Nothing to do here.
#10 Updated by Stefano Kowalke over 2 years ago
- Tags set to typo3flow