Skip to content

Correct readme#37

Open
reloxx13 wants to merge 2 commits into
josegonzalez:masterfrom
reloxx13:patch-2
Open

Correct readme#37
reloxx13 wants to merge 2 commits into
josegonzalez:masterfrom
reloxx13:patch-2

Conversation

@reloxx13

@reloxx13 reloxx13 commented Sep 6, 2018

Copy link
Copy Markdown

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

reloxx13 commented Sep 6, 2018

Copy link
Copy Markdown
Author

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

coveralls commented Sep 6, 2018

Copy link
Copy Markdown

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
Comment thread README.md
$data += [
'user_id' => $user['id'],
$data += [
'modified_by_user_id' => $user->id,

@reloxx13 reloxx13 Sep 7, 2018

Copy link
Copy Markdown
Author

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
Copy Markdown
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