We're updating the issue view to help you get more done. 

on return in prepopulate Plugin SaveItem and EditItem

Description

Prepopulate plugin does not have return in the function of hookSaveItem and hookEditItem so others plugin that try to hook after him won't work.

As a general question
Why that rule has been set? It causes problematic dependence between plugin.

Environment

None

Activity

Show:
avihayh
January 4, 2017, 8:20 AM

How i see it, it is can cause problem if the default is to block. Because new forgot is meaning "take". And more than that today it is not remove the plugins that run before him.
Should be an option that if it is 'true' it will "take" the event.

User known
January 4, 2017, 8:21 AM

Or we can just make sure it's clear that you have to return true. I think that's the real problem. It's not clear.

avihayh
January 4, 2017, 8:44 AM

You are absolutely right,
But how much it is costs to debug it after a forgotten moment.

User known
January 4, 2017, 8:48 AM

Ok we'll fix it!

avihayh
January 4, 2017, 8:49 AM

Thank you!

Fixed

Assignee

User known

Reporter

avihayh

Labels

None

Fix versions

Affects versions

Priority

Major
Configure