On Tue, May 27, 2014 at 10:45 PM, Hugues Peccatte <hugues.peccatte@gmail.com
wrote:
2014-05-27 18:53 GMT+02:00 Marc Delisle marc@infomarc.info:
Hi,
I found out that some bug reports by Scrutinizer are wrong. For example, in [0], the PHP Analyzer reports that you cannot assign a return value because mysqli_init() always returns null, which is not the case.
[0]
https://scrutinizer-ci.com/g/phpmyadmin/phpmyadmin/issues/master/files/libra...
-- Marc Delisle (phpMyAdmin)
Hi,
Another one is this one: https://scrutinizer-ci.com/g/phpmyadmin/phpmyadmin/issues/master/files/libra... Scrutinizer believes that the method isn't doing anything, but it is overwriting $_REQUEST. It seems that Scrutinizer doesn't understand this.
H.
Hi,
There are many more such issues which includes:
1) function duplication in which 2 functions which are completely different are reported as duplicated ones. For e.g. https://scrutinizer-ci.com/g/phpmyadmin/phpmyadmin/indices/152775/duplicatio...
2) https://scrutinizer-ci.com/g/phpmyadmin/phpmyadmin/issues/master/files/libra... The method getMIMEtype cannot be called on $transformation_plugin (of type string). In the if condition it is checked first that $transformation_plugin is of type object and then method getMIMEtype is called.
There are more than one instances of reports which are mentioned above.They need to be reported as false positives.