Skip to content
This repository has been archived by the owner on Oct 20, 2023. It is now read-only.

Fix notice on atomic field #158

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

inigoflores
Copy link

#157 introduced a bug that triggers multiple Notice messages:

Notice (1024): SoftDeleteBehavior::setup(): model PropertyValue has no field!atomic [APP/Plugin/Utils/Model/Behavior/SoftDeleteBehavior.php, line 74]
Notice (1024): SoftDeleteBehavior::setup(): model EmailAddress has no field!atomic [APP/Plugin/Utils/Model/Behavior/SoftDeleteBehavior.php, line 74]
Notice (1024): SoftDeleteBehavior::setup(): model Property has no field!atomic [APP/Plugin/Utils/Model/Behavior/SoftDeleteBehavior.php, line 74]
Notice (1024): SoftDeleteBehavior::setup(): model Contact has no field!atomic [APP/Plugin/Utils/Model/Behavior/SoftDeleteBehavior.php, line 74]

It is easily fixed by removing:

 'atomic' => true,

from the $default array declaration.

This is not needed as, a default value is already set by:

protected $_atomic = true;

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant