Domande taggate «coding-standards»

5
Standard di codifica Magento
Di recente ho iniziato a lavorare ampiamente con SnifferMagento e Code , e vorrei sapere quali standard dovrei seguire. Ho provato gli standard di codifica Zend , ma non voglio usare il limite di 80 righe, e non è nemmeno usato nel core. Dopodiché, ho provato gli standard CS2 , …





5
Ignorando il parametro non utilizzato con lo sniffer di codice
Sto eseguendo il codesniffer con lo standard EcgM2 sulla mia estensione personalizzata e ricevo l'avviso Il parametro del metodo $contextnon viene mai utilizzato per il InstallSchema.phpfile. Come posso eliminare questo avviso? Il mio metodo è simile al seguente (notare la SuppressWarningsparte superiore): /** * {@inheritdoc} * @SuppressWarnings(PHPMD.ExcessiveMethodLength) * @SuppressWarnings(PHPMD.UnusedFormalParameter) */ …

4
Best practice per il filesystem
Sto lavorando su alcune estensioni di Magento 2 che richiedono la lettura di file dal file system. Quando si esegue lo sniffer php utilizzando gli standard ECGM2, si lamenta del fatto che sto usando funzioni come basenameo dirname. L'uso della funzione dirname () è proibito o L'uso della funzione basename …

3
Conformità ECG: come fare?
C'è qualcosa di simile a un coding standardper Magento 1 , chiamato ECGin cui molte cose dovrebbero essere evitate o metodi che non devono essere utilizzati. Proviamo a creare un elenco che copra tutti i casi che generano "errori" o "avvisi" nel codice. Di seguito troverai un elenco con tutti …


1
Cosa stanno controllando i test statici?
Dopo aver creato il mio modulo ho provato a correre bin/magento dev:tests:run static. Ciò ha segnalato alcuni problemi con il mio codice, ma non sono sicuro di cosa sia stato verificato durante l'esecuzione di questi test. La documentazione ufficiale dice L'analisi del codice statico verifica che il codice PHP segua …
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.