Nette coding standards – Změna – od 2.4 nebo 3.0?

praci s prepisovanim nemas :) https://github.com/…ing-standard

Zuben45 napsal(a):

Děkuji, zajímalo mě to hlavně kvůli PR. Osobně si kód rád přizpůsobím Nette coding standardu :)

Jasně, chápu :-)

PR už by se měly automaticky kontrolovat, zda CS vyhovují.

Tomáš Votruba napsal(a):

@Zuben45 Rád ti pomůžu s tou výjimkou. Můžeš založit issue na https://github.com/…ndard/issues

Nejlíp s PHP třídou, na které to failuje. Tu zjistíš tak, že ecs check src specifikuješ na ecs check src/SomeDirectory/SomeFile.php.

Díky @TomášVotruba, jdu zjistit jaký to failuje (v kontrole 495 souborů :)) No když spustím znovu tak pokračuje, takže vždy asi smazat cache ?

EDIT: již vyskakuje i při

vendor/bin/ecs check --clear-cache app/

ale jen občas

EDIT2: @TomášVotruba tak jsem zjistil příčinu, měl jsem v projektu 2 prázdné php soubory :), Ale ty nové řádky jsou opravdu otravné (vždy při fixu se na začátek opravovaných přidá X nových řádku za <?php), v jedné třídě jsem jich měl i přes 1200 :/

Editoval Zuben45 (14. 7. 2017 14:14)

Ty se fixly. Ale souhlas, že mě to taky zmátlo, měla by tam být o tom informace.

Napada nekoho, proc NoWhitespaceInBlankLineFixer mi udela nasledujici „neplechu“ s doc komentari?

Kdyz pouziju fix, tak mi to odsadi dva prazdne radky (viz kode nize), kdy kod znovu zkontroluju, nalezne to dalsi chyby. Kdyz ho nepouziju, nenalezne se zadna chyba.

	# ...and at the end of blank lines.
	- PhpCsFixer\Fixer\Whitespace\NoWhitespaceInBlankLineFixer

Original:
1/1 [============================] 100%
------ ---------------------------------------------------------
Line vendor/aleswita/formrenderer/src/controls\Link.php
------ ---------------------------------------------------------
1 Remove trailing whitespace at the end of blank lines.
(PhpCsFixer\Fixer\Whitespace\NoWhitespaceInBlankLineFixer)
------ ---------------------------------------------------------

class Link extends Nette\Forms\Controls\Button
{
	/**
	 * @param string|object
	 */
	public function __construct($caption = null)
	{
		parent::__construct($caption);
		$this->control = Nette\Utils\Html::el('a');
	}


	/**
	 * @return bool
	 */
	public function isFilled(): bool
	{
		return false;
	}


	/**
	 * @param string|object
	 * @return void
	 */
	public function getLabel($caption = null): void
	{
	}


	/**
	 * @param string|object
	 * @return Nette\Utils\Html
	 */
	public function getControl($caption = null): Nette\Utils\Html
	{
		$this->setOption('rendered', true);
		$el = clone $this->control;

		$el->setText($this->translate($caption === null ? $this->caption : $caption));

		if ($this->isDisabled()) {
			$el->addClass('disabled');
		}

		return $el;
	}
}

After fix:
[OK] 4 errors successfully fixed and no other found!

class Link extends Nette\Forms\Controls\Button
{
	/**
	 * @param string|object
	 */
	public function __construct($caption = null)
	{
		parent::__construct($caption);
		$this->control = Nette\Utils\Html::el('a');
	}


	/**


	 * @return bool
	 */
	public function isFilled(): bool
	{
		return false;
	}


	/**


	 * @param string|object
	 * @return void
	 */
	public function getLabel($caption = null): void
	{
	}


	/**


	 * @param string|object
	 * @return Nette\Utils\Html
	 */
	public function getControl($caption = null): Nette\Utils\Html
	{
		$this->setOption('rendered', true);
		$el = clone $this->control;

		$el->setText($this->translate($caption === null ? $this->caption : $caption));

		if ($this->isDisabled()) {
			$el->addClass('disabled');
		}

		return $el;
	}
}

Ou, sorry, prehlidl jsem 😟