search

OggettoWeb / Oggetto-Coding-Standard

PHP Code Sniffer rules for Oggetto coding standard
4 stars 9 forks source link

fix #6, fix #7 #49

Open rut4 opened 11 years ago

dankocherga commented 10 years ago

Сделал файл с именем test/filenames/foo_bar.php и запустил сниффер:

$ vendor/bin/phpcs --standard=./Oggetto test/filenames/foo_bar.php

На выходе получились вот такие странные ошибки:

FILE: /.../test/filenames/foo_bar.php
--------------------------------------------------------------------------------
FOUND 7 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 1 | ERROR | Character '/' should not be used in file name
 1 | ERROR | Character '/' should not be used in file name
 1 | ERROR | Character '/' should not be used in file name
 1 | ERROR | Character '/' should not be used in file name
 1 | ERROR | Character '/' should not be used in file name
 1 | ERROR | Character '/' should not be used in file name
 1 | ERROR | Character '/' should not be used in file name
--------------------------------------------------------------------------------
dankocherga commented 10 years ago

Также, для валидации строк я бы советовал использовать регулярные выражения, а не проходить циклом по каждому символу.

rut4 commented 10 years ago

Нужно ли указывать какой символ нельзя использовать в имени файла в случае ошибки? Или достаточно просто написать "Invalid file name"?