rhodes 3.3.2.beta.5 → 3.3.2.beta.6

data/CHANGELOG

@@ -1,5 +1,6 @@
 ## 3.3.2
 * iPhone: fix AppStore sumbit issue. Now all files stored in Private Documents, only database and user files in Documents 
+* Fix issues with Ruby 1.9.2 on Mac
 * Android: fix issue with database encryption on OS 4.0
 * Android: disable zooming by default
 * iPhone: fix issue with creating Event

data/Rakefile

@@ -303,6 +303,10 @@ namespace "config" do
       make_application_build_capabilities_header_file
     end
 
+    $rhologhostport = $config["log_host_port"] 
+    $rhologhostport = 52363 unless $rhologhostport
+	$rhologhostaddr = Jake.localip()
+
   end
 
   task :qt do

data/doc/configuration.txt

@@ -113,9 +113,9 @@ Sync server url, typically this will look like 'http://<hostname>:<port>/applica
 
 	syncserver = 'http://localhost:9292/application'
 
-Geo location update period (in seconds). Used as default value for third parameter of GeoLocation.set_notification method. This value is passed to OS API and behaviour is OS dependent (it should be used by system as a tip how often call notification back).
+Geo location update period (in seconds). Used as default value for third parameter of GeoLocation.set_notification method. This value is passed to OS API and behaviour is OS dependent (it should be used by system as a tip how often call notification back). Ignored on iOS.
 
-    gps_ping_timeout_sec = 30
+	gps_ping_timeout_sec = 30
 
 Geo location inactivity timeout (in seconds). This parameter controls the time after that location service is switched off in case no calls to the service. It also used as time period to drop cached location value.
 	

data/doc/device-caps.txt

@@ -199,7 +199,7 @@ Rhodes will update location using network data until the GPS signal has been rec
 There are two ways to use GeoLocation.
 
  * Syncronous call to GeoLocation module for particular value.
- * Set GeoLocation notification and track location by call back. In this case all location values passed to notification as parameters.
+ * Set GeoLocation notification and track location by call back. In this case all location values are passed to notification as parameters.
 
 Note that GeoLocation is switched off automatically when the application goes into the background, and is switched on as the application goes back to foreground.
 However it is possible to switch on location services as usual while the application is in background.
@@ -237,7 +237,7 @@ Test if the location system is up and acquired position:
 	:::ruby
 	GeoLocation.known_position?
 	
-Set callback to track location changes.  
+Set callback to track location changes.
 
 	:::ruby
 	GeoLocation.set_notification(
@@ -245,13 +245,26 @@ Set callback to track location changes.
 
 * callback - callback url; after the callback has been called once, it will be automatically called repeatedly with the current location coordinates passed to the callback as parameters. (You need not reset the callback within the callback.)
 * ping_gpssystem_timeout - optional, if 0 system timeout used, use to redefine timeout in sec for gps system reading
-* callback params: known_position, latitude, longitude, available, status('error', 'ok'), error_code(from RhoError), accuracy
+* callback params: known_position, latitude, longitude, available, status('error', 'ok'), error_code(from RhoError), accuracy(horizontal radius in meters; iOS and Android)
 
-Turn off GeoLocation.
+Also there are two parameters you can set in rhoconfig.txt:
+
+Geo location update period (in seconds). Used as the default value for the third parameter (ping_gpssystem_timeout) of GeoLocation.set_notification method. This value is passed to OS API and behaviour and is OS dependent (it should be used by system as a tip on how often to call notification back). Ignored on iOS.
+
+	gps_ping_timeout_sec = 30
+
+Geo location inactivity timeout (in seconds). This parameter controls the time after that location service is switched off in case there are no calls to the service. It is also used as the time period to drop cached location value.
+	
+	geo_location_inactivity_timeout = gps_ping_timeout_sec * 10
+
+
+Turn off GeoLocation. 
 
 	:::ruby
 	GeoLocation.turnoff
 
+**NOTE: GeoLocation.turnoff - after GPS is switched off, you might still receive a few callbacks (this depends on the platform; iOS does not receive callbacks after turnoff).**
+
 **NOTE: The current behavior of the geolocation callback is that it will be called forever until it is stopped; you need to call `GeoLocation.turnoff` to stop it. Previous behavior was that the callback was called once and needed to be reset.**
 
 Get the distance between two points in miles.
@@ -261,6 +274,8 @@ Get the distance between two points in miles.
 
 GeoCoding: You can use any free web service for direct and reverse geocoding. See our complete example of using Google geocoding web service in [Rhodes-System-Api-Samples example](http://github.com/rhomobile/rhodes-system-api-samples/tree/master/app/GeoCoding).
 
+**NOTE: On iOS, real GPS starts working after the first access to the GeoLocation module. Real GPS hardware is switched off after a call to GeoLocation.turnoff. But keep in mind - any call to GeoLocation, such as getting latitude, starts the hardware GPS again!**
+
 #### Testing GeoLocation
 
 While developing your application on Windows Mobile emulator, you may find the [FakeGPS](http://msdn.microsoft.com/en-us/library/bb158722.aspx) utility useful.
@@ -311,17 +326,14 @@ Callback points to the `geo_callback` in the Settings controller.rb:
     
     def geo_callback
       # navigate to `show_location` page if GPS receiver acquire position  
-	  if @params['known_position'].to_i != 0 && @params['status'] =='ok'	
-	  	WebView.navigate url_for(:action => :show_location) 
-	  end	
-	  # show error if timeout expired and GPS receiver didn't acquire position
-	  if @params['available'].to_i == 0 || @params['status'] !='ok'
-        WebView.navigate url_for(:action => :show_location_error) 
+	  if @params['known_position'].to_i != 0 && @params['status'] =='ok'
+	  
+	  	GeoLocation.set_notification '', '', 2
+	  	WebView.navigate url_for(:action => :map_all)
 	  end
-      # do nothing, still wait for location 
     end
 
-##### Turning off GeoLocation as soon as the app goes of the device front page
+##### Turning off GeoLocation as soon as the app goes off the device front page
 
 In this code sample, /GeoLocation/ has to be replaced with an appropriate regular expression to detect the controller. This code is called from geo_callback. This decision allows you to switch off GeoLocation from a single code point in case geo_callback is set.
 
@@ -336,7 +348,7 @@ In this code sample, /GeoLocation/ has to be replaced with an appropriate regula
 	  end
 	end
 
-If the app does not need location updates, and GeoLocation should still remian active, you can send an empty notification.
+If the app does not need location updates, and GeoLocation should still remain active, you can send an empty notification.
 
 	:::ruby
 	GeoLocation.set_notification "", "", 30
@@ -1621,10 +1633,11 @@ Before Rhodes 3.3.2 all files stored in /Documents/ but Apple require do not pla
 Now all files except databases stored in /Library/Caches/Private Documents/ - files in this folder do not bakup.
 If you want make any files should be bakup - use "user folder" for it - Rho::RhoApplication::get_user_path. All you files created in old version in Rho::RhoApplication::get_base_app_path now should be open in Rho::RhoApplication::get_user_path.
 
-Also there are few additional parameters in build.yml (use it only if you want tune iOS specific folder scheme for some reason):
-* iphone_db_in_approot - if 1 then place databases files into <rhodes root>
-* iphone_set_approot - set <rhodes root> to one of the three folders (property value - folder): "Documents" - /Documents/, "Library_Caches" - /Library/Caches/Private Documents/, "Library_Private_Documents" - /Library/Private Documents/
-* iphone_userpath_in_approot  - is 1 then <rhodes user root> will be in the same place with <rhodes root>
+Also there are few additional parameters in build.yml (use it only if you want tune iOS specific folder scheme for some reason).
+
+ * iphone_db_in_approot - if 1 then place databases files into <rhodes root>
+ * iphone_set_approot - set <rhodes root> to one of the three folders (property value - folder): "Documents" - /Documents/, "Library_Caches" - /Library/Caches/Private Documents/, "Library_Private_Documents" - /Library/Private Documents/
+ * iphone_userpath_in_approot  - is 1 then <rhodes user root> will be in the same place with <rhodes root>
 
 ### change low level parameters in build.yml example
 	iphone_db_in_approot: 1

data/doc/ui.txt

@@ -591,6 +591,30 @@ For the tabbar:
 	  end
 	end
 
+Also you can use Hash instead of Array for additional parameters (the same with call runtime functionality) :
+	:::ruby
+	class AppApplication < Rho::RhoApplication
+	  def initialize
+	    # Tab items are loaded left->right, @tabs[0] is leftmost tab in the tab-bar
+	    @tabs = {
+	      :background_color => 0x0000FF,
+	      :tabs => [
+	      { :label => "Dashboard", :action => '/app', 
+			:icon => "/public/images/tabs/dashboard.png", :reload => true, :web_bkg_color => 0x7F7F7F }, 
+	      { :label => "Accounts",  :action => '/app/Account',  
+			:icon => "/public/images/tabs/accounts.png" },
+	      { :label => "Contacts",  :action => '/app/Contact',  
+			:icon => "/public/images/tabs/contacts.png" },
+	      { :label => "Options",   :action => '/app/Settings', 
+			:icon => "/public/images/tabs/options.png" }
+	      ]
+	    }
+	    # Important to call super _after_ you define @tabs!
+	    super
+	  end
+	end
+
+
 Each tabbar item defined in the above sample defines the following tab elements:
 
 * :label - Visible label to display on the tabbar (required)

data/lib/build/jake.rb

@@ -141,8 +141,8 @@ class Jake
   end
 
   def self.run_local_server(port = 0)
-    addr = localip
-    server = WEBrick::HTTPServer.new :BindAddress => addr, :Port => port
+    addr = localip                   #:BindAddress => addr,
+    server = WEBrick::HTTPServer.new :Port => port
     port = server.config[:Port]
     puts "LOCAL SERVER STARTED ON #{addr}:#{port}"
     Thread.new { server.start }
@@ -574,6 +574,89 @@ class Jake
       Dir.chdir currentdir
 
   end
+  
+  def self.run_rho_log_server(app_path)
+
+	confpath_content = File.read($srcdir + "/apps/rhoconfig.txt") if File.exists?($srcdir + "/apps/rhoconfig.txt")
+	confpath_content += "\r\n" + "rhologhost=" + $rhologhostaddr
+	confpath_content += "\r\n" + "rhologport=" + $rhologhostport.to_s()
+	File.open($srcdir + "/apps/rhoconfig.txt", "w") { |f| f.write(confpath_content) }  if confpath_content && confpath_content.length()>0
+  
+    begin
+        require 'net/http'
+        
+        res = Net::HTTP.start(Jake.localip(), $rhologhostport) {|http|
+             http.post('/', "RHOLOG_GET_APP_NAME")
+        }
+        puts "res : #{res}"
+        puts "body : #{res.body}"
+        
+        if ( res && res.body == app_path)
+            puts "Log server is already running. Reuse it."
+            
+	        started = File.open($app_path + "/started", "w+")
+	        started.close
+            
+            return
+        else
+            puts "Close Log server for another app."            
+            res = Net::HTTP.start(Jake.localip(), $rhologhostport) {|http|
+                 http.post('/', "RHOLOG_CLOSE")
+            }
+            
+        end
+    rescue Exception => e
+        puts "EXC: #{e}"
+    end    
+  
+    system("START rake run:webrickrhologserver[#{app_path}]")
+  end
     
 end
   
+namespace :run do
+  desc "start rholog(webrick) server"
+  task :webrickrhologserver, :app_path  do |t, args|
+    puts "Args were: #{args}"
+    $app_path = args[:app_path]
+    
+    Rake::Task["config:wm"].invoke
+
+	$rhologserver = WEBrick::HTTPServer.new :Port => $rhologhostport
+	
+	puts "LOCAL SERVER STARTED ON #{$rhologhostaddr}:#{$rhologhostport}"
+	started = File.open($app_path + "/started", "w+")
+	started.close
+	
+	#write host and port 4 log server     
+	$rhologfile = File.open(getLogPath, "w+")
+	
+	$rhologserver.mount_proc '/' do |req,res|
+	    if ( req.body == "RHOLOG_GET_APP_NAME" )
+		    res.status = 200
+		    res.chunked = true
+		    res.body = $app_path
+		elsif ( req.body == "RHOLOG_CLOSE" )
+		    res.status = 200
+		    res.chunked = true
+		    res.body = ""
+		
+		    $rhologserver.shutdown
+	    else
+		    $rhologfile.puts req.body
+		    $rhologfile.flush
+		    res.status = 200
+		    res.chunked = true
+		    res.body = ""
+		end    
+	end
+
+    ['INT', 'TERM'].each {|signal| 
+        trap(signal) {$rhologserver.shutdown}
+    }
+	
+	$rhologserver.start
+	$rhologfile.close
+	
+  end
+end

data/lib/framework/rho/rhoapplication.rb

@@ -79,7 +79,13 @@ module Rho
       if @@native_bar_data[:type] == :vtabbar
         tabs = @@native_bar_data[:data]
         # normalize the list
-        tabs.map! { |tab| tab[:refresh] = false unless tab[:refresh]; tab }
+        if tabs.kind_of? Array
+             tabs.map! { |tab| tab[:refresh] = false unless tab[:refresh]; tab }
+        else
+             if tabs.kind_of? Hash
+                 tabs[:tabs].map! { |tab| tab[:refresh] = false unless tab[:refresh]; tab }
+             end
+        end
         puts "Initializing application with vertical tabs: #{tabs.inspect}"
         NativeTabbar.create_vertical(tabs)
         #NativeTabbar.create(tabs)
@@ -87,7 +93,13 @@ module Rho
       elsif @@native_bar_data[:type] == :tabbar
         tabs = @@native_bar_data[:data]
         # normalize the list
-        tabs.map! { |tab| tab[:refresh] = false unless tab[:refresh]; tab }
+        if tabs.kind_of? Array
+             tabs.map! { |tab| tab[:refresh] = false unless tab[:refresh]; tab }
+        else
+             if tabs.kind_of? Hash
+                  tabs[:tabs].map! { |tab| tab[:refresh] = false unless tab[:refresh]; tab }
+             end
+        end
         puts "Initializing application with tabs: #{tabs.inspect}"
         NativeTabbar.create(tabs)
         NativeTabbar.switch_tab(0)

data/platform/android/Rhodes/res/layout/datetime.xml

@@ -2,11 +2,11 @@
 
 <ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
 	android:orientation="vertical" android:layout_width="fill_parent"
-	android:layout_height="fill_parent" android:fillViewport="true">
-<LinearLayout android:id="@+id/datetime" android:layout_width="fill_parent" android:layout_height="fill_parent" xmlns:android="http://schemas.android.com/apk/res/android" android:layout_gravity="center_horizontal" android:orientation="vertical">
-<DatePicker android:layout_height="wrap_content" android:id="@+id/datePicker" android:layout_gravity="top|center_horizontal" android:layout_width="wrap_content"></DatePicker>
-<TimePicker android:layout_height="wrap_content" android:id="@+id/timePicker" android:layout_gravity="center_vertical|center_horizontal" android:layout_width="wrap_content"></TimePicker>
-<LinearLayout android:id="@+id/LinearLayout01" android:layout_weight="1" android:layout_gravity="center_vertical|center_horizontal" android:layout_height="wrap_content" android:layout_width="wrap_content">
+	android:layout_height="wrap_content" android:fillViewport="true" android:layout_gravity="center_vertical">
+<LinearLayout android:id="@+id/datetime" android:layout_width="wrap_content" android:layout_height="wrap_content" xmlns:android="http://schemas.android.com/apk/res/android" android:layout_gravity="center_vertical|center_horizontal" android:orientation="vertical">
+<DatePicker android:layout_height="wrap_content" android:id="@+id/datePicker" android:layout_gravity="center_vertical|center_horizontal" android:layout_width="wrap_content" android:layout_margin="7dp"></DatePicker>
+<TimePicker android:layout_height="wrap_content" android:id="@+id/timePicker" android:layout_gravity="center_vertical|center_horizontal" android:layout_width="wrap_content" android:layout_marginBottom="7dp" android:visibility="visible"></TimePicker>
+<LinearLayout android:id="@+id/LinearLayout01" android:layout_gravity="center_vertical|center_horizontal" android:layout_height="wrap_content" android:layout_width="wrap_content" android:layout_margin="20dp">
 <Button android:layout_width="wrap_content" android:layout_height="wrap_content" android:layout_gravity="center_vertical" android:id="@+id/okButton" android:text="@string/ok" android:clickable="true"></Button><Button android:layout_width="wrap_content" android:layout_height="wrap_content" android:layout_gravity="center_vertical" android:text="@string/cancel" android:id="@+id/cancelButton" android:clickable="true"></Button>
 </LinearLayout>
 </LinearLayout>

data/platform/android/Rhodes/src/com/rhomobile/rhodes/datetime/DateTimePickerScreen.java

@@ -314,10 +314,10 @@ public class DateTimePickerScreen extends BaseActivity {
 
 		switch (_fmt) {
 		case FORMAT_DATE:
-			_timePicker.setVisibility(View.INVISIBLE);
+			_timePicker.setVisibility(View.GONE);
 			break;
 		case FORMAT_TIME:
-			_datePicker.setVisibility(View.INVISIBLE);
+			_datePicker.setVisibility(View.GONE);
 			break;
 		case FORMAT_DATE_TIME:
 			break;

data/platform/android/Rhodes/src/com/rhomobile/rhodes/mapview/AnnotationsOverlay.java

@@ -38,6 +38,7 @@ import com.google.android.maps.GeoPoint;
 import com.google.android.maps.ItemizedOverlay;
 import com.google.android.maps.OverlayItem;
 import com.rhomobile.rhodes.file.RhoFileApi;
+import com.rhomobile.rhodes.util.PerformOnUiThread;
 
 public class AnnotationsOverlay extends ItemizedOverlay<OverlayItem> {
 	
@@ -121,10 +122,15 @@ public class AnnotationsOverlay extends ItemizedOverlay<OverlayItem> {
 	
 	@Override
 	protected boolean onTap(int i) {
-		Annotation ann = annotations.elementAt(i);
+		final Annotation ann = annotations.elementAt(i);
 		//if (ann.url == null)
 		//	return false;
-		mainView.selectAnnotation(ann);
+		//PerformOnUiThread.exec(new Runnable() {
+			//public void run() {
+				mainView.selectAnnotation(ann);
+			//}
+		//}, false);
+		
 		//WebView.navigate(ann.url, WebView.activeTab());
 		//mainView.finish();
 		return true;

data/platform/android/Rhodes/src/com/rhomobile/rhodes/mapview/CalloutOverlay.java

@@ -55,88 +55,152 @@ import com.rhomobile.rhodes.util.Utils;
 
 public class CalloutOverlay extends ItemizedOverlay<OverlayItem> {
 	
-	public static final String TAG = "AnnotationOverlay";
+	public static final String TAG = "CalloutOverlay";
 
 	private GoogleMapView mainView = null;
 	private Callout mCallout = null; 
 	private Annotation mSelectedAnnotation = null;
 	
+	
+	private Boolean is_logging = false;
+	
+	private void printLog(String msg) {
+		if (is_logging) {
+			Utils.platformLog(TAG, msg);
+		}
+	}
+	
 	public CalloutOverlay(GoogleMapView view, Drawable marker) {
 		super(boundCenterBottom(marker));
 		mainView = view;
+		mSelectedAnnotation = null;
 		populate();
 	}
 	
 	public void selectAnnotation(Annotation ann) {
-		mSelectedAnnotation = ann;
+		printLog("selectAnnotation() START");
+		
 		if (mCallout == null) {
 			mCallout = new Callout(ann.latitude, ann.longitude, ann.title, ann.subtitle, ann.url, mainView);
 		}
 		else {
 			mCallout.rebuild(ann.latitude, ann.longitude, ann.title, ann.subtitle, ann.url);
 		}
-		populate();
+		printLog("selectAnnotation() prePopulate");
+		final Annotation _ann = ann;
+		//PerformOnUiThread.exec(new Runnable() {
+			Annotation mAnn = _ann;
+			//public void run() {
+				mSelectedAnnotation = mAnn;
+				populate();
+				//mainView.
+			//}
+		//}, false);
+		printLog("selectAnnotation() FINISH");
 	}
 	
 	public void deselectAnnotation() {
-		mSelectedAnnotation = null;
+		printLog("deselectAnnotation() START");
+		printLog("deselectAnnotation() prePopulate");
+		//PerformOnUiThread.exec(new Runnable() {
+			//public void run() {
+				mSelectedAnnotation = null;
+				populate();
+			//}
+		//}, false);
+		printLog("deselectAnnotation() FINISH");
 	}
 	
 	
 	@Override
 	protected OverlayItem createItem(int i) {
+		printLog("  --  createItem() START");
 		Annotation ann = mSelectedAnnotation;
-		GeoPoint pnt = new GeoPoint((int)(ann.latitude*1000000), (int)(ann.longitude*1000000));
-		OverlayItem item = new OverlayItem(pnt, ann.title, ann.subtitle);
-
-		Bitmap bitmap = mCallout.getResultBitmap();
-		bitmap.setDensity(DisplayMetrics.DENSITY_MEDIUM);
-		BitmapDrawable bd = new BitmapDrawable(bitmap);
-		//bd.setTargetDensity(DisplayMetrics.DENSITY_MEDIUM);
-		bd.setVisible(true, true);
-		bd.setBounds(	ann.callout_x_offset + mCallout.getXOffset(), 
-						ann.callout_y_offset + mCallout.getYOffset(), 
-						ann.callout_x_offset + mCallout.getXOffset() + bitmap.getWidth(), 
-						ann.callout_y_offset + mCallout.getYOffset() + bitmap.getHeight());
+		OverlayItem item = null;
 		
-		item.setMarker(bd);
+		if (ann != null) {
+			GeoPoint pnt = new GeoPoint((int)(ann.latitude*1000000), (int)(ann.longitude*1000000));
+			item = new OverlayItem(pnt, ann.title, ann.subtitle);
+	
+			Bitmap bitmap = mCallout.getResultBitmap();
+			bitmap.setDensity(DisplayMetrics.DENSITY_MEDIUM);
+			BitmapDrawable bd = new BitmapDrawable(bitmap);
+			//bd.setTargetDensity(DisplayMetrics.DENSITY_MEDIUM);
+			bd.setVisible(true, true);
+			bd.setBounds(	ann.callout_x_offset + mCallout.getXOffset(), 
+							ann.callout_y_offset + mCallout.getYOffset(), 
+							ann.callout_x_offset + mCallout.getXOffset() + bitmap.getWidth(), 
+							ann.callout_y_offset + mCallout.getYOffset() + bitmap.getHeight());
+			
+			item.setMarker(bd);
+		}
+		else {
+			GeoPoint pnt = new GeoPoint((int)(0), (int)(0));
+			item = new OverlayItem(pnt, "", "");
+			Bitmap bm = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_8888);
+			bm.eraseColor(0);
+			bm.setDensity(DisplayMetrics.DENSITY_MEDIUM);
+			BitmapDrawable bd = new BitmapDrawable(bm);
+			bd.setVisible(true, true);
+			bd.setBounds(	0, 
+							0, 
+							1, 
+							1);
+			
+			item.setMarker(bd);
+		}
+		printLog("  --  createItem() FINISH");
 		return item;
 	}
 
 	@Override
 	public int size() {
-		if (mSelectedAnnotation != null) {
-			return 1;
-		}
-		return 0;
+		int s = 0;
+		//if (mSelectedAnnotation != null) {
+			s = 1;
+		//}
+		printLog("size() == "+String.valueOf(s));
+		return s;
 	}
 	
 	@Override
 	protected boolean onTap(int i) {
+		printLog("onTap("+String.valueOf(i)+") START");
 		if (mSelectedAnnotation == null) {
+			printLog("onTap() return false");
 			return false;
 		}
 		if ((mSelectedAnnotation.url != null) && (mSelectedAnnotation.url.length() > 0)) {
 			WebView.navigate(mSelectedAnnotation.url, WebView.activeTab());
 			mainView.finish();
+			printLog("onTap() return true 1");
 			return true;
 		}
+		printLog("onTap() return true 2");
 		return true;
 	}
 
 	@Override
 	public boolean onTap(GeoPoint p, com.google.android.maps.MapView mapView) {
+		printLog("onTap() START");
 		boolean result = super.onTap(p, mapView);
+		printLog("onTap() after super");
 		if (!result) {
 			if (mSelectedAnnotation != null) {
-				PerformOnUiThread.exec(new Runnable() {
+				deselectAnnotation();
+				/*
+				 PerformOnUiThread.exec(new Runnable() {
 					public void run() {
-						mSelectedAnnotation = null;
-						populate();
+						//mSelectedAnnotation = null;
+						//populate();
+						deselectAnnotation();
 					}
 				}, false);
+				//*/
+				
 			}
 		}
+		printLog("onTap() FINISH");
 		return result;
 	}