Update cookbook/security/acl.rst

[DenisGorbachev]

See doctrine/DoctrineMongoDBBundle#4 (comment)

[iampersistent]

👍

[jmikola]

Additionally, you should probably stick to the existing column wrap (the link will likely go on its own line).

[DenisGorbachev]

Like this?

[jmikola]

Question for @iampersistent: although you depend on the ODM bundle, you technically only require a connection from doctrine/mongodb, correct? That's the library analogous to Doctrine DBAL. If so, I suggest we rewrite the paragraph like so:

    The ACL system requires a connection from either Doctrine DBAL (usable by
    default) or Doctrine MongoDB (usable with `MongoDBAclBundle`_). However,
    that does not mean that you have to use Doctrine ORM or ODM for mapping your
    domain objects. You can use whatever mapper you like for your objects, be it
    Doctrine ORM, MongoDB ODM, Propel, raw SQL, etc. The choice is yours.

Most doc chapters define external links at the bottom of the file, so I would also suggest adding this towards the end (see doctrine.rst for an existing example):

.. _`MongoDBAclBundle`: https://github.com/IamPersistent/MongoDBAclBundle

[iampersistent]

@jmikola you are correct. In fact, it shouldn't have any dependency on the bundle. That was just some carelessness on my part when I pulled the code from the ODM Bundle. I've removed it and update the dependency to doctrine/mongodb. Thanks for catching that.

[DenisGorbachev]

Should I reflect this in PR?

On Wed, Sep 12, 2012 at 8:01 AM, Richard Shank [email protected]:

@jmikola https://github.com/jmikola you are correct. In fact, it
shouldn't have any dependency on the bundle. That was just some
carelessness on my part when I pulled the code from the ODM Bundle. I've
removed it and update the dependency to doctrine/mongodb. Thanks for
catching that.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1697#issuecomment-8481383.

Денис Горбачев,
Руководитель коллектива «Быстрее ветра»
http://www.faster-than-wind.ru/
+7 919 968-00-40

[jmikola]

@iampersistent It looks like MongoDBAclBundle is still depending on ODM: https://github.com/iampersistent/MongoDBAclBundle/blob/master/composer.json

[weaverryan]

Hey guys!

I've merged this in and pulled in the rewording from @jmikola

Thanks!

[iampersistent]

@jmikola I'll look at it this evening