Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct readme #37

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Correct readme #37

wants to merge 2 commits into from

Conversation

reloxx13
Copy link

@reloxx13 reloxx13 commented Sep 6, 2018

The Readme said add the Behavior in the ENTITY, but it has to be in the TABLE.

Added info for fields option.

Also small md fix.

add behavior in table class not entity.


add option fields description
@reloxx13
Copy link
Author

reloxx13 commented Sep 6, 2018

Then define an event listener to handle the event and pass in additional metadata, for example:

this is very bad described, where do i have to place the eventlistener (folder).
im sure ill find it in the cakephp doc tomorrow,, but it would save time to know it while reading. im done for now :)

@coveralls
Copy link

coveralls commented Sep 6, 2018

Coverage Status

Coverage remained the same at 78.634% when pulling e95287e on reloxx13:patch-2 into 10ad5a7 on josegonzalez:master.

Add some more Listener Hints
$data += [
'user_id' => $user['id'],
$data += [
'modified_by_user_id' => $user->id,
Copy link
Author

@reloxx13 reloxx13 Sep 7, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im not sure about this, i use a custom auth and in $this->Auth->user() is an array with my oauth tokendata and the user entity.

so its not the default footprint data

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you change the example?

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

Successfully merging this pull request may close these issues.

3 participants