Input data loss on saveReady on Publish

[Vayu Robins]

Hi.

I am hooking into the saveReady hook in order to update data on other pages, when a page is published. In other words, when a transaction is published, the user is updated. Certain fields are reset on Publish for some reason. It's only on publish, so if I insert data into input fields again and press save, then it works.

So i have added this bit of code to the admin.php file in my template.

$page->addHook( 'Pages::saveReady', null, 'credit_published' );
function credit_published( $event ) {
	$page = $event->arguments[0];

	if( $page->template == "credit-log" && $page->id && !$page->isTrash() && $page->isChanged( 'status' ) && !$page->is( Page::statusUnpublished ) && $page->credit_action->id === 3 ) {
		// Add credit to user.
		$partner = $page->partner;
		$partner->of( false );
		$partner->credit = intval( $partner->credit + $page->credit );
		$partner->save();
	}
}

[Soma]

You don't need to save when hooking saveReady as PW does it already right after hook.

[Vayu Robins]

Thanks Soma, but I need to update another page (a partner page) when a transaction page is published. I use $partner->save() to do this. If uncomment this, then the problem disappears. What could be wrong with the code?

[LostKobrakai]

How about using the Pages::published hook?

You don't need to save when hooking saveReady as PW does it already right after hook.

He's saving "another" page, so nope, it's needed there.

[Vayu Robins]

In the input field called "Partner" I get the user. I then save the user into the variable $partner. I then get the amount stored in the input field called "Kredit". I add the credit to the users credit.

[Soma]

A save will uncache all pages and you loose data.. so maybe not use save ready but after save.

[Vayu Robins]

Soma, is that the $page->addHookAfter( 'Pages::saved', null, 'credit_published' ); you refer to?

LostKobraKai, is that the $page->addHookAfter( 'Page::published', null, 'credit_published' ); you refer to?

I will try.

[LostKobrakai]

Soma, is that the $page->addHookAfter( 'Pages::saved', null, 'credit_published' ); you refer to?

LostKobraKai, is that the $page->addHookAfter( 'Page::published', null, 'credit_published' ); you refer to?

I will try.

It's Pages::published. I've corrected it already in my previous post, but it seems you didn't notice.

[Vayu Robins]

Thank you very much. So far so good... I can't check for status change the same way, but I think maybe I don't need that here.

[LostKobrakai]

Nope, the hook is explicitly only called if a page is getting published.

[Vayu Robins]

That should perfect!