ZF2-75: Fixes CS for some abstract classes (classname, and location)

Issue Type: Coding Standards Violation Created: 2011-10-01T07:52:41.000+0000 Last Updated: 2012-07-09T15:30:55.000+0000 Status: Resolved Fix version(s): Reporter: Benoît Durand (intiilapa) Assignee: Maks 3w (maks3w) Tags: - Zend\I18n

  • Zend\Ldap
  • Zend\Pdf
  • Zend\Tag

Related issues: Attachments:


source :…

{quote} Abstract Classes

Abstract classes follow the same conventions as classes, with one additional rule: abstract class names MUST begin with the term, "Abstract". As examples, "AbstractAdapter" and "AbstractWriter" are both considered valid abstract class names.

Abstract classes SHOULD be in the same namespace as concrete implementations. {quote}

<pre class="highlight">
$ grep -r "abstract class" library/ | grep -v Abstract | grep -v " \* "

Matthew or Ralph, can you add Zend\Mvc in components list please?


Posted by Thomas Weidner (thomas) on 2011-10-03T20:38:04.000+0000

Zend_Translator changed with GH-460

Posted by Frank Brückner (frosch) on 2012-02-26T11:19:55.000+0000

Zend_Navigation changed with GH-823

Posted by Maks 3w (maks3w) on 2012-03-14T12:03:00.000+0000

I will fix this for Zend\Ldap component.

Posted by Frank Brückner (frosch) on 2012-05-22T10:57:49.000+0000

Zend_Form removed as affected component.

Posted by Maks 3w (maks3w) on 2012-05-23T15:22:15.000+0000

I close this issue in favor of the open story in AgileZen (#52)

Have you found an issue?

See the Overview section for more details.


© 2006-2016 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.