DI zavislot ve factory $this ->isAjax()

Martk napsal(a):

Jestli je ajax request se v prezenteru zjišťuje takto: https://github.com/…resenter.php#… , takže by mělo stačit injektnout Nette\Http\IRequest

Edit: Jestli chceš přepisovat snippety, tak to doporučuji až v presenteru:

protected function createComponentMenuPage(string $name): DataGrid
{
	return $this->ArticleMenuGridFactory->create($this->id, function () {
		if ($this->isAjax()) {
			$this->redrawControl('snippet');
		}
	});
}

public function statusChange($id, $new_status)
{
	$this->getSelection2()->where('id',$id)->update(['active' => $new_status]);

 	($this->afterAction)();
}

Potrebuji rozeznat jak fce byla pouzita treba u statusChange bude $this[‚menuPage‘]->redrawItem($id); aby se me provedl spravnej pozadavek. jestli mi rozumis

Martk napsal(a):

Třeba takto, konstanta není potřeba, když to použiješ jen u této metody

event:

class AfterActionEvent
{

	private const STATUS_CHANGE_TYPE = 'statusChange';

	/** @var string */
	private $type;

	/** @var array */
	private $arguments;

	public function __construct(string $type, array $arguments = [])
	{
		$this->type = $type;
		$this->arguments = $arguments;
	}

	public function isType(string $type): bool
	{
		return $this->type === $type;
	}

	public function getArgument(string $name)
	{
		if (!isset($this->arguments[$name])) {
			throw new InvalidArgumentException("Argument $name does not exist");
		}

		return $this->arguments[$name];
	}

}

public function statusChange($id, $new_status)
{
	$this->getSelection2()->where('id',$id)->update(['active' => $new_status]);

 	($this->afterAction)(new AfterActionEvent(AfterActionEvent::STATUS_CHANGE_TYPE, ['id' => $id]));
}

protected function createComponentMenuPage(string $name): DataGrid
{
	return $this->ArticleMenuGridFactory->create($this->id, function (AfterActionEvent $event) {
		if ($this->isAjax() && $event->isType(AfterActionEvent::STATUS_CHANGE_TYPE)) {
			$this->redrawControl($event->getArgument('id'));
		}
	});
}

Jde to čistějším způsobem, ale na mě to je teď moc psaní.

<?php
public function statusChange($id, $new_status)
{
	$this->getSelection2()->where('id',$id)->update(['active' => $new_status]);

 	($this->afterAction)(new AfterActionEvent(AfterActionEvent::STATUS_CHANGE_TYPE, ['id' => $id]));
}
?>

upravil jsem dle tvych rad a chyba, ale nechapu kde.

<?php
/** @var string*/
    private $afterAction;
.
.
($this->afterAction)(new AfterActionEvent(AfterActionEvent::STATUS_CHANGE_TYPE, ['id' => $id]));/*chyba*/
?>

a hlasi me to chybu Function name must be a string

Editoval turbo80 (2. 7. 2020 15:45)

Martk napsal(a):

Proměnná $this->afterAction nebude validní callback, vsadím se, že je null namísto předávaného closure

je pravda ze jsem ji definoval private ajako string nic vic

@turbo80 Martk se te snazil navest, abys do afterAction ulozil ten callback, ktery se posila z presenteru. za me je lepsi reseni nepouzivat tu property a udrzet si factory bezstavovovu (muze to zpusobit problemy) a provest to zhruba takhle

1. do promenne si muzes ulozit closure, v presenteru tak treba muzes zapsat

$doSomething = function () {
	if ($this->isAjax()) {
...
	}
}

2. tuhle funkci si muzes poslat do tovarny.
treba tvoje tovarna na grid prijima id, tak bude prijimat i callback

public function create($id, $someCallback)
{

a v createComponent nejaky ten callback predas

a v kdekoliv ten callback muzes zavolat

$someCallback();

nebo si do nej predat argumenty

$someCallback($id)

3. a muzes ho vyuzit v nejakym callbacku komponenty, treba

$grid->onChange[] = function () use ($someCallback) {
	...
	$someCallback();
}