Titanium

[ALOY-761] BlackBerry: Map annotation are not displayed if object properties are set in the view .xml

GitHub Issuen/a
TypeBug
PriorityHigh
StatusClosed
ResolutionFixed
Resolution Date2014-08-06T22:14:09.000+0000
Affected Version/sAlloy 1.2.0
Fix Version/sAlloy 1.2.0, 2013 Sprint 15
ComponentsTitanium SDK
Labelsqe-closed-3.1.2, qe-manualtest
ReporterFederico Casali
AssigneeTony Lukasavage
Created2013-07-25T23:24:32.000+0000
Updated2014-08-06T22:14:09.000+0000

Description

Problem description

Map annotation is now shown on the Map if the Annotation properties are set in the View .xml file. If set in the .tss file, annotation is correctly visible.

Steps to reproduce

1. See sample code attached. 2. On BlackBerry, only the annotations with the properties only set in the index.tss file are visible.
<Annotation title="Palo Alto" latitude="37.47" longitude="-122.12"/>

and also the Required 

<Require src="annotationView"/>
are not visible, while
<Annotation id="annotation1"/>

and

<Annotation id="annotation2" platform="blackberry"/>
are correctly displayed.

Attachments

FileDateSize
maps.zip2013-07-25T23:24:32.000+00004264492
Resources.zip2013-07-26T00:49:27.000+00004305135

Comments

  1. Tony Lukasavage 2013-07-26 That's odd as there is special parsing code in annotations to convert the XML string attribute to a float. It solves the same problem on Android, which also does not accept strings as values for latitude and longitude. I'll have to investigate the generated code. [~fcasali] would it be possible for you to show me the generated code for the working and non-working version?
  2. Federico Casali 2013-07-26 I've attached the generated Resource folder for the provided code in the sample and attached to the Jira. Both working and not-working annotations are in there.
  3. Russell McMahon 2013-07-26 Tony we could fix it on the platform side but if Alloy is already doing the conversion that is probably better.
  4. Tony Lukasavage 2013-07-26 [~rmcmahon] That's the confusing part, I'm already doing the conversion for all platforms. The API is defined as a float value in the documentation, so I would define this as Alloy's responsibility.
  5. Tony Lukasavage 2013-07-26 PR: https://github.com/appcelerator/alloy/pull/197 Functional testing can be confirmed with the steps you used to produce the error before.
  6. Federico Casali 2013-08-10 Verified fixed. Titanium SDK 3.1.2.v20130808180613 Alloy 1.2.0-alpha6 Appcelerator Studio 3.1.2.201308082014 CLI 3.1.2-alpha Node 0.10.13 Closing.
  7. Federico Casali 2014-08-06 Reopening for updating 'tester' field, labels and log time.

JSON Source