etobiNov 14, 2024
Wir räumen eine recht große legacy code base auf. Rector hilft uns dabei sehr, macht aber auch den CodeStyle durcheinander. phpcs kann das meiste sehr gut wieder aufräumen - nur große arrays nicht (TCA, ext_emconf). Rector schreibt alles items in eine laaaaaange Zeile.
#typo3 #phpcs #rector #phpcsfixer #codestyle
Show threadetobi
PHPStrom kann das hübsch machen, aber ich finde keine Rule für eins der üblichen Tools, so dass wir das automatisieren und in der CI testen können…
Ich kann mir nicht vorstellen dass es da NICHTS gibt… gucke ich falsch? Hat jemand einen Tipp?
#typo3 #phpcs #rector #phpcsfixer #codestyle
Nov 14, 2024 at 6:34amWeb
Show threadRobert Weißgraeber🇪🇺Nov 14, 2024
@etobi mal chatgpt probiert?
Show threadetobiNov 14, 2024
@rw007 für diese spezielle Frage nein.
Für andere Dinge (dokumentiere das, passt das zusammen) schon. Aber immer mit einem schlechten Gewissen: wie lange muss jetzt Sonne auf meine PV Anlage scheinen, um diese Frage zu beantworten und mir eine halbe Stunde zu sparen
Show threadJulian HofmannNov 14, 2024
Wie groß ist der volkswirtschaftliche Schaden, der durch schlechter lesbaren Code mittel- und langfristig entsteht, wenn Du die Frage nicht löst? Bzw wie hoch, indem Du andere von ihrer Arbeit abhältst? 🤔
(Helfe Dir weiterhin gerne)
@etobi @rw007
Show threadOliver KleeNov 14, 2024
@etobi PHP_Codesniffer kann lange Zeilen umformatieren, wenn ich mich nicht irre. Bin mir aber nicht ganz sicher.
Show threadetobiNov 14, 2024
@oliklee habs leider noch nicht hinbekommen
Show threadUlli ☘️Nov 14, 2024
@etobi im Linting verwenden wir https://packagist.org/packages/editorconfig-checker/editorconfig-checker. Eine lange Zeile ist in Sachen Einrückung aber wahrscheinlich schon valide.
editorconfig-checker/editorconfig-checker - Packagist

A tool to verify that your files follow the rules of your .editorconfig

Show threadetobiNov 14, 2024
@ulli meckern über zu lange Zeile macht auch der phpcs und php-cs-fixer. Aber fixen kann das keiner (nur phpstorm)
Show threadSimon PraetoriusNov 14, 2024

@etobi Sowas tut nicht?

https://github.com/squizlabs/PHP_CodeSniffer/blob/master/src/Standards/Generic/Sniffs/Arrays/ArrayIndentSniff.php

PHP_CodeSniffer/src/Standards/Generic/Sniffs/Arrays/ArrayIndentSniff.php at master · squizlabs/PHP_CodeSniffer

PHP_CodeSniffer tokenizes PHP files and detects violations of a defined set of coding standards. - squizlabs/PHP_CodeSniffer

GitHub
Show threadJulian HofmannNov 14, 2024
Die processSingleLineArray() sieht nicht sonderlich vielversprechend aus ;-)
@s2b @etobi
Show threadSimon PraetoriusNov 14, 2024
@Xitnelat War ja klar, dass du nur den ersten Funktionsnamen liest. Du ChatGPT? 😄 @etobi
Show threadetobiNov 14, 2024
@s2b der fixt Einrückungen innerhalb des Arrays. Und phpcs meckert auch eine zu lange Zeile an, aber die kann er nicht fixen.