Skip to content

Commit

Permalink
Merge pull request #635 from tidev/230504_toColor
Browse files Browse the repository at this point in the history
fix(android): use activity in toColor
  • Loading branch information
cb1kenobi committed May 13, 2023
2 parents 2f033b3 + c20183a commit d0bde93
Show file tree
Hide file tree
Showing 5 changed files with 34 additions and 19 deletions.
2 changes: 1 addition & 1 deletion android/manifest
Expand Up @@ -2,7 +2,7 @@
# this is your module manifest and used by Titanium
# during compilation, packaging, distribution, etc.
#
version: 5.5.1
version: 5.5.2
apiversion: 4
architectures: arm64-v8a armeabi-v7a x86 x86_64
description: External version of Map module using native Google Maps library
Expand Down
13 changes: 9 additions & 4 deletions android/src/ti/map/CircleProxy.java
Expand Up @@ -6,6 +6,7 @@
*/
package ti.map;

import android.app.Activity;
import android.graphics.Color;
import android.os.Message;
import android.view.ViewGroup;
Expand Down Expand Up @@ -146,6 +147,7 @@ public boolean handleMessage(Message msg)
public void processOptions()
{
options = new CircleOptions();
Activity currentActivity = TiApplication.getAppCurrentActivity();

if (hasProperty(MapModule.PROPERTY_CENTER)) {
options.center(TiMapUtils.parseLocation(getProperty(MapModule.PROPERTY_CENTER)));
Expand All @@ -160,11 +162,13 @@ public void processOptions()
}

if (hasProperty(MapModule.PROPERTY_STROKE_COLOR)) {
options.strokeColor(alphaColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_STROKE_COLOR))));
options.strokeColor(
alphaColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_STROKE_COLOR), currentActivity)));
}

if (hasProperty(MapModule.PROPERTY_FILL_COLOR)) {
options.fillColor(alphaColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_FILL_COLOR))));
options.fillColor(
alphaColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_FILL_COLOR), currentActivity)));
}

if (hasProperty(MapModule.PROPERTY_ZINDEX)) {
Expand All @@ -184,6 +188,7 @@ public void processOptions()
public void onPropertyChanged(String name, Object value)
{
super.onPropertyChanged(name, value);
Activity currentActivity = TiApplication.getAppCurrentActivity();
if (circle == null) {
return;
}
Expand All @@ -203,12 +208,12 @@ else if (name.equals(MapModule.PROPERTY_STROKE_WIDTH)) {

else if (name.equals(MapModule.PROPERTY_STROKE_COLOR)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_STROKE_COLOR),
TiConvert.toColor((String) value));
TiConvert.toColor((String) value, currentActivity));
}

else if (name.equals(MapModule.PROPERTY_FILL_COLOR)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_FILL_COLOR),
TiConvert.toColor((String) value));
TiConvert.toColor((String) value, currentActivity));
}

else if (name.equals(MapModule.PROPERTY_ZINDEX)) {
Expand Down
15 changes: 9 additions & 6 deletions android/src/ti/map/PolygonProxy.java
Expand Up @@ -6,6 +6,7 @@
*/
package ti.map;

import android.app.Activity;
import android.os.Message;
import com.google.android.gms.maps.model.LatLng;
import com.google.android.gms.maps.model.Polygon;
Expand All @@ -18,6 +19,7 @@
import org.appcelerator.kroll.annotations.Kroll;
import org.appcelerator.kroll.common.AsyncResult;
import org.appcelerator.kroll.common.TiMessenger;
import org.appcelerator.titanium.TiApplication;
import org.appcelerator.titanium.TiC;
import org.appcelerator.titanium.util.TiConvert;
import ti.map.Shape.IShape;
Expand Down Expand Up @@ -109,7 +111,7 @@ public void processOptions()
{

options = new PolygonOptions();

Activity currentActivity = TiApplication.getAppCurrentActivity();
if (hasProperty(MapModule.PROPERTY_POINTS)) {
processPoints(getProperty(MapModule.PROPERTY_POINTS), false);
}
Expand All @@ -119,15 +121,16 @@ public void processOptions()
}

if (hasProperty(MapModule.PROPERTY_STROKE_COLOR)) {
options.strokeColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_STROKE_COLOR)));
options.strokeColor(
TiConvert.toColor((String) getProperty(MapModule.PROPERTY_STROKE_COLOR), currentActivity));
}

if (hasProperty(MapModule.PROPERTY_STROKE_WIDTH)) {
options.strokeWidth(TiConvert.toFloat(getProperty(MapModule.PROPERTY_STROKE_WIDTH)));
}

if (hasProperty(MapModule.PROPERTY_FILL_COLOR)) {
options.fillColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_FILL_COLOR)));
options.fillColor(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_FILL_COLOR), currentActivity));
}

if (hasProperty(MapModule.PROPERTY_ZINDEX)) {
Expand Down Expand Up @@ -266,7 +269,7 @@ public void onPropertyChanged(String name, Object value)
{

super.onPropertyChanged(name, value);

Activity currentActivity = TiApplication.getAppCurrentActivity();
if (polygon == null) {
return;
}
Expand All @@ -282,12 +285,12 @@ else if (name.equals(MapModule.PROPERTY_POINTS)) {

else if (name.equals(MapModule.PROPERTY_STROKE_COLOR)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_STROKE_COLOR),
TiConvert.toColor((String) value));
TiConvert.toColor((String) value, currentActivity));
}

else if (name.equals(MapModule.PROPERTY_FILL_COLOR)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_FILL_COLOR),
TiConvert.toColor((String) value));
TiConvert.toColor((String) value, currentActivity));
}

else if (name.equals(MapModule.PROPERTY_ZINDEX)) {
Expand Down
14 changes: 9 additions & 5 deletions android/src/ti/map/PolylineProxy.java
Expand Up @@ -6,6 +6,7 @@
*/
package ti.map;

import android.app.Activity;
import android.os.Message;
import com.google.android.gms.maps.model.Dash;
import com.google.android.gms.maps.model.Dot;
Expand All @@ -24,6 +25,7 @@
import org.appcelerator.kroll.common.AsyncResult;
import org.appcelerator.kroll.common.Log;
import org.appcelerator.kroll.common.TiMessenger;
import org.appcelerator.titanium.TiApplication;
import org.appcelerator.titanium.TiC;
import org.appcelerator.titanium.util.TiConvert;
import ti.map.Shape.IShape;
Expand Down Expand Up @@ -134,6 +136,7 @@ public void processOptions()
{

options = new PolylineOptions();
Activity currentActivity = TiApplication.getAppCurrentActivity();
// (int) strokeColor
// (float) strokeWidth
// (int) fillColor
Expand All @@ -144,12 +147,13 @@ public void processOptions()
}

if (hasProperty(MapModule.PROPERTY_STROKE_COLOR)) {
options.color(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_STROKE_COLOR)));
options.color(TiConvert.toColor((String) getProperty(MapModule.PROPERTY_STROKE_COLOR), currentActivity));
}

// alternate API
if (hasProperty(PolylineProxy.PROPERTY_STROKE_COLOR2)) {
options.color(TiConvert.toColor((String) getProperty(PolylineProxy.PROPERTY_STROKE_COLOR2)));
options.color(
TiConvert.toColor((String) getProperty(PolylineProxy.PROPERTY_STROKE_COLOR2), currentActivity));
}

if (hasProperty(MapModule.PROPERTY_STROKE_WIDTH)) {
Expand Down Expand Up @@ -243,7 +247,7 @@ public void onPropertyChanged(String name, Object value)
{

super.onPropertyChanged(name, value);

Activity currentActivity = TiApplication.getAppCurrentActivity();
if (polyline == null) {
return;
}
Expand All @@ -264,12 +268,12 @@ else if (name.equals(PolylineProxy.PROPERTY_STROKE_WIDTH2)) {

else if (name.equals(MapModule.PROPERTY_STROKE_COLOR)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_STROKE_COLOR),
TiConvert.toColor((String) value));
TiConvert.toColor((String) value, currentActivity));
}
// alternate API
else if (name.equals(PolylineProxy.PROPERTY_STROKE_COLOR2)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_STROKE_COLOR),
TiConvert.toColor((String) value));
TiConvert.toColor((String) value, currentActivity));
}

else if (name.equals(PolylineProxy.PROPERTY_ZINDEX)) {
Expand Down
9 changes: 6 additions & 3 deletions android/src/ti/map/RouteProxy.java
Expand Up @@ -18,6 +18,7 @@
import org.appcelerator.kroll.annotations.Kroll;
import org.appcelerator.kroll.common.AsyncResult;
import org.appcelerator.kroll.common.TiMessenger;
import org.appcelerator.titanium.TiApplication;
import org.appcelerator.titanium.TiC;
import org.appcelerator.titanium.util.TiConvert;

Expand Down Expand Up @@ -86,7 +87,8 @@ public void processOptions()
}

if (hasProperty(TiC.PROPERTY_COLOR)) {
options.color(TiConvert.toColor((String) getProperty(TiC.PROPERTY_COLOR)));
options.color(
TiConvert.toColor((String) getProperty(TiC.PROPERTY_COLOR), TiApplication.getAppCurrentActivity()));
}
}

Expand Down Expand Up @@ -160,8 +162,9 @@ else if (name.equals(MapModule.PROPERTY_POINTS)) {
}

else if (name.equals(TiC.PROPERTY_COLOR)) {
TiMessenger.sendBlockingMainMessage(getMainHandler().obtainMessage(MSG_SET_COLOR),
TiConvert.toColor((String) value));
TiMessenger.sendBlockingMainMessage(
getMainHandler().obtainMessage(MSG_SET_COLOR),
TiConvert.toColor((String) value, TiApplication.getAppCurrentActivity()));
}

else if (name.equals(TiC.PROPERTY_WIDTH)) {
Expand Down

0 comments on commit d0bde93

Please sign in to comment.