You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Jan Piotrowski (janpio) (JIRA)" <ji...@apache.org> on 2019/05/24 21:31:00 UTC

[jira] [Commented] (CB-5105) remove dead code for device.version from native plugin source

    [ https://issues.apache.org/jira/browse/CB-5105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16847910#comment-16847910 ] 

Jan Piotrowski (janpio) commented on CB-5105:
---------------------------------------------

Might be a bit of a stretch, but does somebody remember what "iOS is doing something special, but doing it right, so don't make any changes to iOS." referred to?
 

> remove dead code for device.version from native plugin source
> -------------------------------------------------------------
>
>                 Key: CB-5105
>                 URL: https://issues.apache.org/jira/browse/CB-5105
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: cordova-plugin-device
>    Affects Versions: 3.1.0
>            Reporter: Marcel Kinard
>            Assignee: Martin Gonzalez
>            Priority: Minor
>
> The solution for CB-5085 (and related) is for device.js to set device.cordova using CORDOVA_JS_BUILD_LABEL from cordova.js, instead of having native code do that (usually with hardcoded values) and return it back from exec(). So once CB-5085 is fixed in this way, then any native code that computes device.version is dead. That dead code should be removed from cordova-plugin-device, if for no other reason than reducing confusion.
> The following platforms are affected: Android, BB10, WP8, Win8.
> iOS is doing something special, but doing it right, so don't make any changes to iOS.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org