vianetz | We do Magento®

Häufig gestellte Fragen

Falls Sie weitere Fragen haben, kontaktieren Sie mich bitte gerne.


Welchen Score erhält die AdvancedInvoiceLayout Magento-Extension durch das Tool "Judge"?

Letzte Akt.: 28.09.2014

Die Firma Netresearch GmbH hat mit Judge ein Tool entwickelt, dass Magento-Extensions analysiert und mit einem Score bewertet. Der Score ist allerdings für die Analyse von kostenfreien Modulen entwickelt, so dass wir hier die Ausgabe des Tools nach der Analyse unserer AdvancedInvoiceLayout Extension vorstellen möchten.
Prinzipiell sind die ermittelten Scores/Prüfergebnisse mit etwas Vorsicht zu lesen, da natürlich kleinere, unkomplexere Extensions hier etwas bevorzugt werden (z.B. was die Anzahl von Rewrites angeht, etc.).

Ausgabe des Judge-Tools

Skipping plugin "MageCompatibility"
Skipping plugin "SourceCodeComplexity"
Vendor: Vianetz
Extension: AdvancedInvoiceLayout
Version: 2.0.11
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check CoreHacks
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check Rewrites
* rewrite: Found 4 model rewrite(s):
  * core/email_template_mailer => Vianetz_AdvancedInvoiceLayout_Model_Email_Template_Mailer
  * sales/order_pdf_invoice => Vianetz_AdvancedInvoiceLayout_Model_Rewrite_Order_Pdf_Invoice
  * sales/order_pdf_shipment => Vianetz_AdvancedInvoiceLayout_Model_Rewrite_Order_Pdf_Shipment
  * sales/order_pdf_creditmemo => Vianetz_AdvancedInvoiceLayout_Model_Rewrite_Order_Pdf_Creditmemo
* rewrite: Found 1 controller rewrite(s):
  * Frontend area - Mage_Sales => Vianetz_AdvancedInvoiceLayout
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check CodeRuin
* warning: Fix request "@todo"
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check PhpCompatibility
* php_compatibility: Extension is compatible to PHP from version 5.1.2 up to latest versions
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check CheckStyle
* error: Line exceeds maximum limit of 120 characters
* error: Expected 1 space before logical operator
* error: Spaces must be used to indent lines
* error: Line indented incorrectly
* error: Expected 5 space(s) before asterisk
* error: Space found before comma in function call
* error: Opening brace indented incorrectly
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check CheckComments
* cloc_to_ncloc: 0.86
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check CodeCoverage
* unfinished: Failed to execute CodeCoverage plugin.
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check SecurityCheck
* avoidable_calls: Avoidable statement "require_once" usage
Extensions src/app/code/community/Vianetz/AdvancedInvoiceLayout have passed the check PerformanceCheck

Erklärung der einzelnen Checks:

  • Die Checks "MageCompatibility" und "SourceCodeComplexity" konnten leider noch nicht getestet werden, da diese zu Fehlern des Tools führten.
  • Die AdvancedInvoiceLayout-Extension verwendet insgesamt 4 Model Rewrites:
    • Die Mailer-Komponente muss überschrieben werden, um das Anhängen der Email-Attachments zu realisieren. Hierzu gibt es leider keine bessere Alternative. Sofern man diese Funktionalität nicht nutzt, kann der Rewrite aber entfernt werden
    • Die restlichen 3 Model-Rewrites werden aus Kompatibilitätsgründen gesetzt, damit z.B. andere 3rd-Party-Extensions die originalen PDF-Models weiterhin nutzen können, aber die AdvancedInvoiceLayout-Extension greift. Auch diese können ohne Probleme entfernt werden, wenn man diese Funktionalität nicht benötigt.
  • Der Controller-Rewrite im Frontend ist für den Download der PDF-Rechnung im Magento-Benutzerkonto notwendig, um bestehende Routen nutzen zu können. Auch hier gilt, dass der Rewrite ggfs. entfernt werden kann.
  • Eine Erklärung des "cloc_to_ncloc"-Wertes findet sich z.B. auf Wikipedia. Er gibt das Verhältnis von nicht kommentierten Code-Zeilen zu kommentierten an.

« Zurück zur FAQ-Übersicht