Details
-
Type:
Coding Standards Violation
-
Status:
Resolved
-
Priority:
Trivial
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 1.10.0
-
Component/s: Zend_Acl, Zend_Amf, Zend_Application, Zend_Auth, Zend_Auth_Adapter_Ldap, Zend_Auth_Adapter_OpenId, Zend_Cache, Zend_Captcha, Zend_CodeGenerator, Zend_Config, Zend_Config_Writer, Zend_Console_Getopt, Zend_Controller, Zend_Crypt, Zend_Db, Zend_Db_Adapter_Db2, Zend_Db_Adapter_Mysqli, Zend_Db_Adapter_Oracle, Zend_Db_Profiler, Zend_Db_Select, Zend_Db_Table, Zend_Debug, Zend_Dojo, Zend_Dom_Query, Zend_Feed, Zend_Feed_Reader, Zend_File_Transfer, Zend_Filter, Zend_Filter_Inflector, Zend_Filter_Input, Zend_Form, Zend_Gdata, Zend_Http_Client, Zend_Http_Cookie, Zend_Http_CookieJar, Zend_Http_Response, Zend_Http_Server, Zend_InfoCard, Zend_Json, Zend_Json_Server, Zend_Layout, Zend_Ldap, Zend_Loader, Zend_Locale, Zend_Log, Zend_Mail, Zend_Mail_Storage, Zend_Measure, Zend_Memory, Zend_Mime, Zend_Navigation, Zend_OpenId, Zend_Paginator, Zend_Pdf, Zend_ProgressBar, Zend_Reflection, Zend_Registry, Zend_Rest_Client, Zend_Rest_Server, Zend_Search_Lucene, Zend_Server_Reflection, Zend_Service_Akismet, Zend_Service_Amazon, Zend_Service_Amazon_Ec2, Zend_Service_Audioscrobbler, Zend_Service_Delicious, Zend_Service_Flickr, Zend_Service_Nirvanix, Zend_Service_ReCaptcha, Zend_Service_Simpy, Zend_Service_SlideShare, Zend_Service_StrikeIron, Zend_Service_Technorati, Zend_Service_Twitter, Zend_Service_Yahoo, Zend_Session, Zend_Soap_Client, Zend_Soap_Server, Zend_Soap_Wsdl, Zend_Tag, Zend_Test_PHPUnit, Zend_Text_Figlet, Zend_Text_Table, Zend_TimeSync, Zend_Tool, Zend_Translate, Zend_Uri, Zend_Validate, Zend_Validate_File, Zend_Version, Zend_View, Zend_Wildfire, Zend_XmlRpc_Client, Zend_XmlRpc_Server, ZendX_Console_Process_Unix, ZendX_JQuery, ZendX_Whois
-
Labels:None
Description
I will add them if necessary.
Currently we have this page: http://framework.zend.com/wiki/display/ZFDEV/Subversion+Standards#SubversionStandards-TipSVNautoprops
If it's important, perhaps we could add it into coding standards.
Issue Links
| This issue is related to: | ||||
| ZF-9029 | Multiple files in library/ or in tests/ don't have the svn:keywords=id |
|
|
|
Activity
Hide
Robin Skoglund
added a comment -
Point taken. I can see now how it might be useful in some situations. It shouldn't be necessary for unit tests, though, as Pádraic states.
I also agree with Pádraic on the 'elseif' / 'else if' struct, but that's another issue 
Show
Robin Skoglund
added a comment - Point taken. I can see now how it might be useful in some situations. It shouldn't be necessary for unit tests, though, as Pádraic states.
I also agree with Pádraic on the 'elseif' / 'else if' struct, but that's another issue
In my opinion, ZF should stop mixing SCM properties in files. I find it weird, clumsy, and unnecessary that each file in a repository has a line that says when the file was last updated in the particular source control repository the file is tied to at the moment. This information is available in the SCM itself, and should not have to be part of the actual file contents.