feat(dev-tools): Mise en place phpcs+phpstan (#4829) (90be9985) · Validations · spip / plan

.gitignore

0 → 100644

+7 −0

+10 −10

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -51,13 +51,13 @@ function action_deplacer_objets_dist() {
		}

		// ne modifier que si les emplacements n'ont pas déjà changé !
		$ids = sql_allfetsel($_id_table, $table, array(sql_in($_id_table, $ids), $champ . '=' . sql_quote($id_rubrique_old)));
		$ids = sql_allfetsel($_id_table, $table, [sql_in($_id_table, $ids), $champ . '=' . sql_quote($id_rubrique_old)]);
		$ids = array_column($ids, $_id_table);

		include_spip('action/editer_objet');

		$errors = $success = array();
		$modifs = array('id_parent' => $id_rubrique_new);
		$errors = $success = [];
		$modifs = ['id_parent' => $id_rubrique_new];

		foreach ($ids as $id) {
		if (autoriser('modifier', $objet, $id)) {
		@@ -77,7 +77,7 @@ function action_deplacer_objets_dist() {
		$ids = sql_allfetsel(
		$_id_table,
		$table,
		array(sql_in($_id_table, $ids), $champ . '=' . sql_quote($id_rubrique_old))
		[sql_in($_id_table, $ids), $champ . '=' . sql_quote($id_rubrique_old)]
		);
		$ids = array_column($ids, $_id_table);
		if ($ids) {
		@@ -88,11 +88,11 @@ function action_deplacer_objets_dist() {
		}
		}

		return plan_json_envoi(array(
		return plan_json_envoi([
		'done' => true,
		'success' => $success,
		'errors' => $errors,
		));
		]);
		}

		function plan_json_envoi($data) {
		@@ -101,9 +101,9 @@ function plan_json_envoi($data) {
		}

		function plan_json_erreur($msg) {
		return plan_json_envoi(array(
		return plan_json_envoi([
		'done' => false,
		'success' => array(),
		'errors' => array($msg)
		));
		'success' => [],
		'errors' => [$msg]
		]);
		}

+2 −2

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -26,12 +26,12 @@ function action_plan_dist() {
		include_spip('inc/utils');
		$objet = table_objet(_request('objet'));

		$env = array(
		$env = [
		'id_rubrique' => intval(_request('id_rubrique')),
		// pour le dépliement d'une rubrique, ne pas lister le contenu récursivement
		'lister' => ($objet == 'rubriques') ? 'rubrique' : 'tout',
		'conteneur' => 'non'
		);
		];

		if ($statut = _request('statut')) {
		$env['statut'] = $statut;

0 → 100644

+7 −0

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		{
		"require-dev": {
		"dealerdirect/phpcodesniffer-composer-installer": "^0.7.1",
		"spip/coding-standards": "^1.2",
		"phpstan/phpstan": "^0.12.98"
		}
		}

0 → 100644

+15 −0

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		<?xml version="1.0"?>
		<ruleset>
		<file>.</file>
		<exclude-pattern>vendor/*</exclude-pattern>
		<exclude-pattern>lang/*</exclude-pattern>
		<exclude-pattern>lib/*</exclude-pattern>

		<rule ref="SPIP41"/>

		<config name="ignore_warnings_on_exit" value="1"/>
		<arg name="cache" value=".php_cs.cache"/>
		<arg name="report-full" value=".php_cs.txt"/>
		<arg name="report-summary"/>
		<arg value="s"/>
		</ruleset>