Ausblick auf die MyOOS [Dumper] Version 5.0.15

[r23]

Hallo,

der Programmcode von MyOOS [Dumper] war schwer lesbar. Ich wollte in den letzten Tagen mal eben die Lesbarkeit verbessern und habe per Hand Leerzeichen usw. eingefügt. mmh...

Ich habe jetzt den Programmcode mit php-cs-fixer optimiert


php-cs-fixer fix ./ -vvv --show-progress=dots

^- Wer sich mit php-cs-fixer auskennt - bitte melden. Evtl. kann man den Programmcode noch weiter verbessern?

Das Projekt ist jetzt ein eigens Projekt bei GitHub viewtopic.php?f=41&t=2950
https://github.com/r23/MyOOS-Dumper

Wunschliste / Künftige Attraktionen

Hast du Verbesserungsvorschläge? Zögere nicht, das Entwicklerteam über das Forum hier zu kontaktieren.

Mitwirken

Wenn du uns dabei helfen möchtest, das MyOOS Projekt zu verbessern, freuen wir uns hier auf deine Pull Requests via GitHub.
https://github.com/r23/MyOOS-Dumper

Beste Grüße

[e-dschungel]

Hi,

^- Wer sich mit php-cs-fixer auskennt - bitte melden. Evtl. kann man den Programmcode noch weiter verbessern?

für Codingstyle-Überprüfung und -Fixes benutze ich PHP Code Sniffer https://github.com/squizlabs/PHP_CodeSniffer. Welcher da leistungsfähiger ist, kann ich aber nicht sagen. Zur statischen Codeanalyse ist phpstan empfehlenswert: https://github.com/phpstan/phpstan
Damit habe ich schon einige interessante Fehler gefunden.

Gruß
e-dschungel

P.S.:
Kleiner Test von PHPStan mit niedrigstem Level nur im inc Verzeichnis:

Code: Alles auswählen

 ------ ------------------------------------------ 
  Line   functions.php                             
 ------ ------------------------------------------ 
  182    Undefined variable: $old_db               
  653    Undefined variable: $tinstall_ftp_server  
 ------ ------------------------------------------ 

 ------ ----------------------------------------------------------------- 
  Line   functions_dump.php                                               
 ------ ----------------------------------------------------------------- 
  296    Function mysqli_select_db invoked with 1 parameter, 2 required.  
  324    Undefined variable: $ommand                                      
  327    Undefined variable: $ommand                                      
 ------ ----------------------------------------------------------------- 

 ------ ------------------------ 
  Line   functions_global.php    
 ------ ------------------------ 
  900    Undefined variable: $s  
 ------ ------------------------ 

 ------ --------------------------------------------------------------------- 
  Line   sqlbrowser/sql_record_insert_inputmask.php                           
 ------ --------------------------------------------------------------------- 
  56     Function FormHiddenParams not found.                                 
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols   
 ------ --------------------------------------------------------------------- 

 ------ --------------------------------------------------------------------- 
  Line   sqlbrowser/sql_record_update_inputmask.php                           
 ------ --------------------------------------------------------------------- 
  60     Function FormHiddenParams not found.                                 
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols   
 ------ --------------------------------------------------------------------- 

 ------ ---------------------------------------------------------------- 
  Line   sqlbrowser/sqlbox.php                                           
 ------ ---------------------------------------------------------------- 
  71     Array has 2 duplicate keys with value 'DBID' ('DBID', 'DBID').  
 ------ ---------------------------------------------------------------- 

 ------ ---------------------------------------------------------------- 
  Line   table_query.php                                                 
 ------ ---------------------------------------------------------------- 
  34     Class MODTemplate referenced with incorrect case: MODtemplate.  
 ------ ---------------------------------------------------------------- 

 ------ ------------------------------------ 
  Line   template.php                        
 ------ ------------------------------------ 
  191    Undefined variable: $lastiteration  
 ------ ------------------------------------ 

                                                                                
 [ERROR] Found 11 errors                    

[r23]

Kleiner Test von PHPStan mit niedrigstem Level nur im inc Verzeichnis:

[ERROR] Found 11 errors [/code]

Danke für den Hinweis.

Ich habe mit der Fehlerbeseitigung begonnen

Schönes Wochenende

Ralf

[e-dschungel]

Super, vielen Dank!