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.