l3pp4rd/DoctrineExtensions — pull request #435 by mgiagnoni (2012-08-22 01:26:04+00:00)
Title: [sluggable] boolean options cannot be set to false with XML mapping

Sluggable configuration options *unique* and *updatable* cannot be set to false when XML mapping is used.

Problem is [here ](https://github.com/l3pp4rd/DoctrineExtensions/blob/master/lib/Gedmo/Sluggable/Mapping/Driver/Xml.php#L79): _getAttribute() always return a string and casting to boolean a string containing a literal 'false' gives true. Proposed fix is creating a method _getBooleanAttribute() in base XML driver class.

Funny thing is that tests should cover this case, but *updatable* is set to false in XML fixture mapping and tested with assertTrue(), so also the test needs to be fixed.

Let me know if you prefer a different solution.