[ALOY-1582] Ti.Platform.osname reports windows
| GitHub Issue | n/a |
|---|
| Type | Bug |
|---|
| Priority | Medium |
|---|
| Status | Closed |
|---|
| Resolution | Fixed |
|---|
| Resolution Date | 2017-09-21T23:14:11.000+0000 |
|---|
| Affected Version/s | Alloy 1.9.14, Alloy 1.10.4 |
|---|
| Fix Version/s | CLI Release 6.3.0 |
|---|
| Components | Runtime |
|---|
| Labels | n/a |
|---|
| Reporter | Ewan Harris |
|---|
| Assignee | Ewan Harris |
|---|
| Created | 2017-09-21T13:03:00.000+0000 |
|---|
| Updated | 2017-10-11T23:05:09.000+0000 |
|---|
Description
Description
When using Ti.Platform.osname on Windows with Alloy, it incorrectly reports windows. The correct value is windowsstore or windowsphone
Steps to reproduce
Add console.log(Ti.Platform.osname); to index.js of an alloy app
Build for Windows
Actual
windows is logged
Expected
windowsstore for ws-local target, windowsphone for wp-emulator target
Comments
JSON Source
Master pr: https://github.com/appcelerator/alloy/pull/851
Validated the fix with this environment: Node Version: 6.10.3 NPM Version: 3.10.10 Windows OS: 10.0.15063.0 Windows Phone: 10.0.15063.0 Appc CLI: 6.3.0-master.14 Appc CLI NPM: 4.2.9 Titanium SDK version:6.2.2