search

AKSW / OntoWiki

Semantic data wiki as well as Linked Data publishing engine
https://ontowiki.net
205 stars 51 forks source link

fixed the errors about empty arrays #352

Closed shinobu closed 8 years ago

white-gecko commented 8 years ago

I still get Notice: Undefined offset: 0 and Notice: Undefined offset: 2 (altering):

Notice: Undefined offset: 0 in …/ontowiki/application/tests/CodeSniffer/Standards/Ontowiki/Sniffs/Commenting/FileCommentSniff.php on line 75

Call Stack:
    0.0001     229880   1. {main}() …/ontowiki/vendor/squizlabs/php_codesniffer/scripts/phpcs:0
    0.0110    2104552   2. PHP_CodeSniffer_CLI->runphpcs() …/ontowiki/vendor/squizlabs/php_codesniffer/scripts/phpcs:25
    0.0113    2162904   3. PHP_CodeSniffer_CLI->process() …/ontowiki/vendor/squizlabs/php_codesniffer/CodeSniffer/CLI.php:106
    0.0355    4469536   4. PHP_CodeSniffer->processFiles() …/ontowiki/vendor/squizlabs/php_codesniffer/CodeSniffer/CLI.php:914
   13.3738   12474272   5. PHP_CodeSniffer->processFile() …/ontowiki/vendor/squizlabs/php_codesniffer/CodeSniffer.php:629
   13.3739   12474536   6. PHP_CodeSniffer->_processFile() …/ontowiki/vendor/squizlabs/php_codesniffer/CodeSniffer.php:1739
   13.3739   12475240   7. PHP_CodeSniffer_File->start() …/ontowiki/vendor/squizlabs/php_codesniffer/CodeSniffer.php:1861
   13.3842   13273496   8. Ontowiki_Sniffs_Commenting_FileCommentSniff->process() …/ontowiki/vendor/squizlabs/php_codesniffer/CodeSniffer/File.php:573
shinobu commented 8 years ago

that's odd, i can't reproduce this error on my pc, did you used the fix/FileCommentSniff branch for testing? neither the git nor the non-git run throw errors for me

white-gecko commented 8 years ago

Yes I'm using the feature branch. Did you enable all PHP error outputs, also for PHP cli?

Natanael

Am 12. Mai 2016 20:06:47 MESZ, schrieb shinobu notifications@github.com:

that's odd, i can't reproduce this error on my pc, did you used the fix/FileCommentSniff branch for testing? neither the git nor the non-git run throw errors for me

You are receiving this because you commented. Reply to this email directly or view it on GitHub: https://github.com/AKSW/OntoWiki/pull/352#issuecomment-218838539

Diese Nachricht wurde von meinem Mobiltelefon mit Linux Kernel und JVM Userland von K-9 Mail gesendet.

shinobu commented 8 years ago

i'm pretty sure it's enabled (i did saw those errors prior to fixing them)

white-gecko commented 8 years ago

Yes I'm on this commit, and get the notices d229668f64d9af29c852d560fcaedf6e113ea847

white-gecko commented 8 years ago 
Notice: Undefined offset: ? in …/ontowiki/application/tests/CodeSniffer/Standards/Ontowiki/Sniffs/Commenting/FileCommentSniff.php on line 71|72|74|75
white-gecko commented 8 years ago

Try to reproduce it by checking out an extension into the extensions folder e.g. https://github.com/AKSW/csvimport.ontowiki

shinobu commented 8 years ago

fixed the error in 24aeae7