Details
-
Type:
Bug
-
Status: Done/Fixed
-
Priority:
Trivial
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.4.6
-
Fix Version/s: 4.5.1
-
Component/s: None
-
Labels:None
-
Documentation Required?:None
Description
Civix generated info.xml don't meet the requirements - I had to add this section to get it to upload as a release.
<urls>
<url desc="Main Extension Page">http://github.com/TechToThePeople/eu.tttp.excel</url>
<url desc="Documentation">http://github.com/TechToThePeople/eu.tttp.excel</url>
<url desc="Support">http://forum.civicrm.org</url><url desc="Licensing">http://civicrm.org/licensing</url>
</urls>
A couple of quasi-related thoughts / nice to haves if it makes sense to look at while delving into civix
1) when I run phpstorm inspect code on civix generated shells I get a few things highlighted like invalid return statements & things. Would be nice if these weren't there as it would encourage better use of IDEs
2) I add a versionAtLeast function to all my extensions to deal with interversion changes - should something like this be in civix / core?
/**
- is version of at least the version provided
* - @param number $version
- @return boolean
*/
function _versionAtLeast($version) {
$codeVersion = explode('.', CRM_Utils_System::version());
if (version_compare($codeVersion[0] . '.' . $codeVersion[1], $version) >= 0) { return TRUE; }return FALSE;
}