Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-15263

civix doesn't add urls to info.xml

    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;
        }

        Attachments

          Activity

            People

            • Assignee:
              timotten Tim Otten
              Reporter:
              eileen Eileen McNaughton
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: