# Code Review des Community Store

**URL:** <https://forum.shopware.com/t/code-review-des-community-store/15001>\
**Category:** Shopware 3.5\
**Tags:** programming\
**Created:** [14. August 2013 um 10:31 UTC](https://forum.shopware.com/t/code-review-des-community-store/15001 "2013-08-14T10:31:14Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![Stxe](https://avatars.discourse-cdn.com/v4/letter/s/edb3f5/32.png) [@Stxe](https://forum.shopware.com/u/Stxe)\
**Post date:** [14. August 2013 um 10:31 UTC](https://forum.shopware.com/t/code-review-des-community-store/15001/1 "2013-08-14T10:31:14Z")

</div>

Hallo Community, ich arbeite jetzt seit einiger Zeit an verschiedensten Shopware-Modulen und zu Ende der Projekte stand ich öfters vor demselben Problem: Beim Upload von Modulen in den Community Store wird ein Code Review ausgelöst, dass trotz Anstrengungen in diesem Bereich gelegentlich Beanstandungen am Modul liefert. Ich frage mich, ob es eine Möglichkeit gibt, das Modul auf dieselben Kriterien zu prüfen, ohne dafür extra einen Upload zu starten. Auf diesem Weg könnte die Qualität im gesamten Entwicklungsprozess überwacht werden, anstatt erst nach Abschluss des Projekts. Ich würde mich über Anregungen sehr freuen. MfG Stxe

---

<div class="post-metadata">

**Author:** ![Stxe](https://avatars.discourse-cdn.com/v4/letter/s/edb3f5/32.png) [@Stxe](https://forum.shopware.com/u/Stxe)\
**Post date:** [28. August 2013 um 15:40 UTC](https://forum.shopware.com/t/code-review-des-community-store/15001/2 "2013-08-28T15:40:56Z")

</div>

\*push\* Gibt es vielleicht jemanden der mir hier weiterhelfen kann? 🙂

---

<div class="post-metadata">

**Author:** ![Stefan\_Hamann](https://avatars.discourse-cdn.com/v4/letter/s/5e9695/32.png) [@Stefan\_Hamann](https://forum.shopware.com/u/Stefan_Hamann)\
**Post date:** [5. September 2013 um 07:25 UTC](https://forum.shopware.com/t/code-review-des-community-store/15001/3 "2013-09-05T07:25:08Z")

</div>

Hey, das ist derzeit nicht so einfach möglich, da die Tests stark auf unsere Infrastruktur abgestimmt sind. Im wesentlichen passiert dort aber kein Hexenwerk: Folgende Dinge werden automatisch überprüft: [list] [\*] Sind Klassen und Methoden korrekt dokumentiert[/\*] [\*] Werden irgendwo Super-Globals wie \_GET / \_REQUEST etc. verwendet -\> Potentielle Sicherheitslücke[/\*] [\*] Ist der PHP-Code syntaktisch korrekt (PHP-Lint)[/\*] [\*] Wird ExtJS verwendet?[/\*] [\*] Ist der Developer-Prefix im Plugin Namespace vorhanden[/\*] [\*] Ist der Klassenname der Bootstrap korrekt[/\*] [\*] Wird bereits ein Plugin unter diesem Namen im Store geführt[/\*] [\*] Ist die Methode getVersion vorhanden[/\*] [\*] Entspricht die übergebene Version der Version die in der Plugin-Verwaltung hinterlegt ist[/\*][/list]

---

<div class="post-metadata">

**Author:** ![Stxe](https://avatars.discourse-cdn.com/v4/letter/s/edb3f5/32.png) [@Stxe](https://forum.shopware.com/u/Stxe)\
**Post date:** [6. September 2013 um 09:10 UTC](https://forum.shopware.com/t/code-review-des-community-store/15001/4 "2013-09-06T09:10:25Z")

</div>

Hi Stefan, Danke, das reicht mir auch als info 🙂 Beste Grüße, Stefan

---

<div class="post-metadata">

**Author:** ![portrino](https://avatars.discourse-cdn.com/v4/letter/p/e9bcb4/32.png) [@portrino](https://forum.shopware.com/u/portrino)\
**Post date:** [2. Februar 2017 um 10:22 UTC](https://forum.shopware.com/t/code-review-des-community-store/15001/5 "2017-02-02T10:22:21Z")

</div>

Wir nutzen verschiedene vendor Packages (von Facebook etc.) die auf SuperGlobals zugreifen? Was sollen wir denn da tun? Ich kann ja nicht einfach deren Code ändern.