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]
-- 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/libraries/DisplayResults.class.php?selectedAuthors[0]=chirayu.chiripal%40gmail.com&orderField=path&order=asc 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. -- Regards, Chirayu Chiripal phpMyAdmin Intern - Google Summer of Code 2014 https://chirayuchiripal.wordpress.com/