for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace Drupal\qa\Views;
use Drupal\qa\Pass;
class Overrides extends Views {
/**
* {@inheritdoc]
*/
public function init() {
$this->package_name = __NAMESPACE__;
$this->title = t('Non-default views');
$this->description = t('Have any views been overridden or only created in the DB ? This is a performance and change management issue.');
}
public function checkViewType($view) {
$status = $view->type == t('Default') ? 1 : 0;
if (!$status) {
$name = empty($view->human_name) ? $view->name : $view->human_name;
$result = [
module_exists('views_ui')
module_exists
If this is a false-positive, you can also ignore this issue in your code via the ignore-call annotation
ignore-call
/** @scrutinizer ignore-call */
? l($name, "admin/structure/views/view/{$view->name}/edit")
l
? /** @scrutinizer ignore-call */ l($name, "admin/structure/views/view/{$view->name}/edit")
: $name,
$view->type
];
else {
$result = NULL;
$ret = [
'name' => $view->name,
'status' => $status,
'result' => $result,
return $ret;
public function run(): Pass {
$pass = parent::run();
$views = views_get_all_views(TRUE);
views_get_all_views
$views = /** @scrutinizer ignore-call */ views_get_all_views(TRUE);
foreach ($views as $view) {
$pass->record($this->checkViewType($view));
$pass->life->end();
// Prepare for theming
$result = [];
// @XXX May be inconsistent with non-BMP strings ?
uksort($pass->result, 'strcasecmp');
foreach ($pass->result as $view_report) {
$result[] = t('!view: @type', [
'!view' => $view_report[0], // Built safe in self::checkViewPhp
'@type' => $view_report[1],
]);
$result = theme('item_list', ['items' => $result]);
theme
$result = /** @scrutinizer ignore-call */ theme('item_list', ['items' => $result]);
$pass->result = $result;
return $pass;
OSInet /
qa
Loading history...