[ 
https://issues.apache.org/jira/browse/CB-5500?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wei Li updated CB-5500:
-----------------------

         Labels: patch  (was: )
    Description: 
When creating a new cordova project using the cli tool, I specified customised 
uri paths for the libs. However, when I ran the platform add command, cordova 
cli is not downloading the lib from the uri I specified. It still tries to 
download from the default url.

The cause of the problem is found in function

has_custom_path

in src/config.js file. Around this line:

if (uri.protocol && uri.protocol[1] ==':')

The value of protocol will be like "http:" or "https:", according to nodejs 
document. 
The value of uri.protocol[1] will never equal to ":", so the above statement 
will always be false.

I think the fix should be like this:

if (uri.protocol && uri.protocol[uri.protocol.length - 1] ==':')

make sure the protocol ends with ":".

A pull request has created to fix this issue:




  was:
When creating a new cordova project using the cli tool, I specified customised 
uri paths for the libs. However, when I ran the platform add command, cordova 
cli is not downloading the lib from the uri I specified. It still tries to 
download from the default url.

The cause of the problem is found in function

has_custom_path

in src/config.js file. Around this line:

if (uri.protocol && uri.protocol[1] ==':')

The value of protocol will be like "http:" or "https:", according to nodejs 
document. 
The value of uri.protocol[1] will never equal to ":", so the above statement 
will always be false.

I think the fix should be like this:

if (uri.protocol && uri.protocol[uri.protocol.length - 1] ==':')

make sure the protocol ends with ":".







> customised lib urls are not used when lazy loading libs 
> --------------------------------------------------------
>
>                 Key: CB-5500
>                 URL: https://issues.apache.org/jira/browse/CB-5500
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: CLI
>    Affects Versions: 3.2.0
>         Environment: All
>            Reporter: Wei Li
>            Priority: Minor
>              Labels: patch
>
> When creating a new cordova project using the cli tool, I specified 
> customised uri paths for the libs. However, when I ran the platform add 
> command, cordova cli is not downloading the lib from the uri I specified. It 
> still tries to download from the default url.
> The cause of the problem is found in function
> has_custom_path
> in src/config.js file. Around this line:
> if (uri.protocol && uri.protocol[1] ==':')
> The value of protocol will be like "http:" or "https:", according to nodejs 
> document. 
> The value of uri.protocol[1] will never equal to ":", so the above statement 
> will always be false.
> I think the fix should be like this:
> if (uri.protocol && uri.protocol[uri.protocol.length - 1] ==':')
> make sure the protocol ends with ":".
> A pull request has created to fix this issue:



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to