Skip to content

Add new docs for the event manager project.#5

Merged
jwage merged 1 commit intomasterfrom
event-manager-docs
Jun 8, 2018
Merged

Add new docs for the event manager project.#5
jwage merged 1 commit intomasterfrom
event-manager-docs

Conversation

@jwage
Copy link
Copy Markdown
Member

@jwage jwage commented Jun 8, 2018

Fixes #4

@jwage jwage added this to the 1.0.0 milestone Jun 8, 2018
@jwage jwage self-assigned this Jun 8, 2018
@jwage jwage added the Documentation A change to the documentation. label Jun 8, 2018
@jwage jwage requested a review from Majkl578 June 8, 2018 01:15
@jwage jwage force-pushed the event-manager-docs branch from 73ada78 to 3079a3f Compare June 8, 2018 01:16
Copy link
Copy Markdown
Contributor

@Majkl578 Majkl578 left a comment

Choose a reason for hiding this comment

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

👍

If this documentation is not helping to answer questions you have about the
Doctrine DBAL, don't panic. You can get help from different sources:

- Gitter chat room `#doctrine/event-manager <https://gitter.im/doctrine/event-manager>`_

This comment was marked as off-topic.

This comment was marked as off-topic.

---------------

The best way to get started is with the :doc:`Introduction <reference/index#introduction>` section
in the documentation. Use the sidebar to browse other documentation for the Doctrine PHP DBAL.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

typo: DBAL

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't see the typo?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry, it's not DBAL, Its event manager.


public function __construct(EventManager $eventManager)
{
$eventManager->addEventListener(array(self::preFoo, self::postFoo), $this);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[] instead of array()


.. code-block:: php

$eventManager->removeEventListener(array(TestEvent::preFoo, TestEvent::postFoo), $testEvent);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[] instead of array()


public function getSubscribedEvents() : array
{
return array(TestEvent::preFoo);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[] instead of array()

@jwage jwage force-pushed the event-manager-docs branch from dda62b1 to 92d662b Compare June 8, 2018 19:32
@jwage jwage merged commit 2606e60 into master Jun 8, 2018
@jwage jwage deleted the event-manager-docs branch June 8, 2018 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation A change to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants