Merge remote-tracking branch 'origin/development' into development

# Conflicts:
#	app/src/main/java/com/jens/automation2/Actions.java

app/src/apkFlavor/java/com/jens/automation2/Rule.java

@@ -348,14 +348,24 @@ public class Rule implements Comparable<Rule>
 
 	public boolean getsGreenLight(Context context)
 	{
-		return applies(context) && hasNotAppliedSinceLastExecution();
+		if(applies(context))
+		{
+			if(hasNotAppliedSinceLastExecution())
+				return true;
+			else
+				Miscellaneous.logEvent("i", "getsGreenLight()", "Rule " + getName() + " has not flipped since its last execution.", 4);
+		}
+		else
+			Miscellaneous.logEvent("i", "getsGreenLight()", "Rule " + getName() + " does not apply.", 4);
+
+		return false;
 	}
 	
 	public boolean applies(Context context)
 	{
 		if(AutomationService.getInstance() == null)
 		{
-			Miscellaneous.logEvent("i", "RuleCheck", "Automation service not running. Rule cannot apply.", 3);
+			Miscellaneous.logEvent("i", "RuleCheck", "Automation service not running. Rule " + getName() + " cannot apply.", 3);
 			return false;
 		}
 		
@@ -393,7 +403,7 @@ public class Rule implements Comparable<Rule>
 
 			if (!found)
 			{
-				Miscellaneous.logEvent("i", String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleCheckOf), this.getName()), String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleDoesntApplyActivityNotPresent), ActivityDetectionReceiver.getDescription(oneTrigger.getActivityDetectionType())), 3);
+				Miscellaneous.logEvent("i", String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleCheckOf), this.getName()), String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleDoesntApplyActivityNotPresent), getName(), ActivityDetectionReceiver.getDescription(oneTrigger.getActivityDetectionType())), 3);
 				return false;
 			}
 			else
@@ -402,7 +412,7 @@ public class Rule implements Comparable<Rule>
 				{
 					if (oneDetectedActivity.getType() == oneTrigger.getActivityDetectionType() && oneDetectedActivity.getConfidence() < Settings.activityDetectionRequiredProbability)
 					{
-						Miscellaneous.logEvent("i", String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleCheckOf), this.getName()), String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleDoesntApplyActivityGivenButTooLowProbability), ActivityDetectionReceiver.getDescription(oneDetectedActivity.getType()), String.valueOf(oneDetectedActivity.getConfidence()), String.valueOf(Settings.activityDetectionRequiredProbability)), 3);
+						Miscellaneous.logEvent("i", String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleCheckOf), this.getName()), String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.ruleDoesntApplyActivityGivenButTooLowProbability), getName(), ActivityDetectionReceiver.getDescription(oneDetectedActivity.getType()), String.valueOf(oneDetectedActivity.getConfidence()), String.valueOf(Settings.activityDetectionRequiredProbability)), 3);
 						return false;
 					}
 				}
@@ -534,15 +544,7 @@ public class Rule implements Comparable<Rule>
 	public void activate(AutomationService automationService, boolean force)
 	{
 		ActivateRuleTask task = new ActivateRuleTask();
-		
-//		if(Settings.startNewThreadForRuleActivation)
 		task.execute(automationService, force);
-//		else
-//		{
-//			task.activateInternally(automationService, force);
-//			AutomationService.updateNotification();
-//			ActivityMainScreen.updateMainScreen();
-//		}		
 	}
 	
 	public static ArrayList<Rule> findRuleCandidatesByPoi(PointOfInterest searchPoi, boolean triggerParameter)
@@ -627,7 +629,7 @@ public class Rule implements Comparable<Rule>
 					
 					if(oneTrigger.getTimeFrame().getTriggerTimeStart().getTime() > oneTrigger.getTimeFrame().getTriggerTimeStop().getTime())
 					{
-						Miscellaneous.logEvent("i", "Timeframe search", "Rule goes over midnight.", 5);
+						Miscellaneous.logEvent("i", "Timeframe search", "Rule (" + oneRule.getName() + ") stretches over midnight.", 5);
 						if(oneTrigger.getTimeFrame().getTriggerTimeStart().getTime() <= searchTime.getTime() || searchTime.getTime() <= oneTrigger.getTimeFrame().getTriggerTimeStop().getTime()+20000) //add 20 seconds because of delay
 						{
 							ruleCandidates.add(oneRule);
@@ -636,7 +638,7 @@ public class Rule implements Comparable<Rule>
 					}
 					else if(oneTrigger.getTimeFrame().getTriggerTimeStart().getTime() <= searchTime.getTime() && searchTime.getTime() <= oneTrigger.getTimeFrame().getTriggerTimeStop().getTime()+20000) //add 20 seconds because of delay
 					{
-						Miscellaneous.logEvent("i", "RuleSearch", "Rule found with TimeFrame with time " + searchTime.toString(), 3);
+						Miscellaneous.logEvent("i", "RuleSearch", "Rule found (" + oneRule.getName() + ") with TimeFrame with time " + searchTime.toString(), 3);
 						ruleCandidates.add(oneRule);
 						break innerloop; //if the poi is found we don't need to search the other triggers in the same rule
 					}

app/src/main/java/com/jens/automation2/Action.java

@@ -5,6 +5,7 @@ import android.os.AsyncTask;
 import android.util.Log;
 import android.widget.Toast;
 
+import org.apache.commons.lang3.StringUtils;
 import org.apache.http.client.methods.HttpGet;
 
 import java.util.ArrayList;
@@ -429,7 +430,10 @@ public class Action
 				case turnScreenOnOrOff:
 					if(getParameter1())
 					{
+						if(StringUtils.isNumeric(this.getParameter2()))
 							Actions.wakeupDevice(Long.parseLong(this.getParameter2()));
+						else
+							Actions.wakeupDevice((long)1000);
 						// wakeupDevice() will create a separate thread. That'll take some time, we wait 100ms.
 						try
 						{

app/src/main/java/com/jens/automation2/Actions.java

@@ -1124,7 +1124,7 @@ public class Actions
 		// turn on screen
 		Miscellaneous.logEvent("i", "Actions", "Turning screen on.", 3);
 		PowerManager pm = (PowerManager) context.getSystemService(Context.POWER_SERVICE);
-		WakeLock wakeLock = pm.newWakeLock(PowerManager.SCREEN_BRIGHT_WAKE_LOCK | PowerManager.ACQUIRE_CAUSES_WAKEUP, "tag");
+		WakeLock wakeLock = pm.newWakeLock(PowerManager.SCREEN_BRIGHT_WAKE_LOCK | PowerManager.ACQUIRE_CAUSES_WAKEUP, AutomationService.NOTIFICATION_CHANNEL_ID + ":turnOffScreen");
 		wakeLock.acquire();
 	}
 
@@ -1141,6 +1141,9 @@ public class Actions
 		getWindow().setAttributes(params);
 
 		PowerManager pm = (PowerManager) context.getSystemService(Context.POWER_SERVICE);
+		WakeLock wakeLock = pm.newWakeLock(PowerManager.PROXIMITY_SCREEN_OFF_WAKE_LOCK,AutomationService.NOTIFICATION_CHANNEL_ID + ":turnOffScreen");
+//		WakeLock wakeLock = pm.newWakeLock(PowerManager.SCREEN_DIM_WAKE_LOCK , AutomationService.NOTIFICATION_CHANNEL_ID + ":turnOffScreen");
+		wakeLock.acquire();
 		WakeLock wakeLock = pm.newWakeLock(PowerManager.PROXIMITY_SCREEN_OFF_WAKE_LOCK, "tag");
 //		WakeLock wakeLock = pm.newWakeLock(PowerManager.SCREEN_DIM_WAKE_LOCK , "tag");
 		wakeLock.acquire();*/

app/src/main/java/com/jens/automation2/ActivityManageRule.java

@@ -49,6 +49,7 @@ import java.util.Collections;
 public class ActivityManageRule extends Activity
 {
 	final static String activityDetectionClassPath = "com.jens.automation2.receivers.ActivityDetectionReceiver";
+	public final static String intentNameTriggerParameter1 = "triggerParameter1";
 
 	public Context context;
 	private Button cmdTriggerAdd, cmdActionAdd, cmdSaveRule;
@@ -265,6 +266,7 @@ public class ActivityManageRule extends Activity
 						break;
 					case devicePosition:
 						Intent devicePositionEditor = new Intent(ActivityManageRule.this, ActivityManageTriggerDevicePosition.class);
+						devicePositionEditor.putExtra(ActivityManageRule.intentNameTriggerParameter1, selectedTrigger.getTriggerParameter());
 						devicePositionEditor.putExtra(ActivityManageTriggerDevicePosition.vectorFieldName, selectedTrigger.getTriggerParameter2());
 						startActivityForResult(devicePositionEditor, requestCodeTriggerDevicePositionEdit);
 						break;
@@ -1365,6 +1367,7 @@ public class ActivityManageRule extends Activity
 			{
 //				newTrigger.setTriggerParameter(data.getBooleanExtra("wifiState", false));
 				newTrigger.setTriggerParameter2(data.getStringExtra(ActivityManageTriggerDevicePosition.vectorFieldName));
+				newTrigger.setParentRule(ruleToEdit);
 				ruleToEdit.getTriggerSet().add(newTrigger);
 				this.refreshTriggerList();
 			}
@@ -1375,7 +1378,9 @@ public class ActivityManageRule extends Activity
 			{
 				Trigger editedTrigger = new Trigger();
 				editedTrigger.setTriggerType(Trigger_Enum.devicePosition);
+				editedTrigger.setTriggerParameter(data.getBooleanExtra(ActivityManageRule.intentNameTriggerParameter1, true));
 				editedTrigger.setTriggerParameter2(data.getStringExtra(ActivityManageTriggerDevicePosition.vectorFieldName));
+				editedTrigger.setParentRule(ruleToEdit);
 				ruleToEdit.getTriggerSet().set(editIndex, editedTrigger);
 				this.refreshTriggerList();
 			}

app/src/main/java/com/jens/automation2/ActivityManageTriggerDevicePosition.java

@@ -9,6 +9,7 @@ import android.text.Spanned;
 import android.util.Log;
 import android.view.View;
 import android.widget.Button;
+import android.widget.CheckBox;
 import android.widget.EditText;
 import android.widget.TextView;
 import android.widget.Toast;
@@ -24,6 +25,7 @@ public class ActivityManageTriggerDevicePosition extends Activity
     TextView currentAzimuth, currentPitch, currentRoll, tvAppliesAzimuth, tvAppliesPitch, tvAppliesRoll;
     Button bApplyPositionValues, bSavePositionValues;
     EditText etDesiredAzimuth, etDesiredAzimuthTolerance, etDesiredPitch, etDesiredPitchTolerance, etDesiredRoll, etDesiredRollTolerance;
+    CheckBox chkDevicePositionApplies;
 
     public static String vectorFieldName = "deviceVector";
 
@@ -37,11 +39,11 @@ public class ActivityManageTriggerDevicePosition extends Activity
         currentPitch.setText(Float.toString(pitch));
         currentRoll.setText(Float.toString(roll));
 
-        if(checkInputs(false))
+        try
         {
             desiredAzimuth = Float.parseFloat(etDesiredAzimuth.getText().toString());
             desiredAzimuthTolerance = Float.parseFloat(etDesiredAzimuthTolerance.getText().toString());
-            if(Math.abs(azimuth) <= Math.abs(desiredAzimuth - desiredAzimuthTolerance) || Math.abs(azimuth) <= desiredAzimuth + desiredAzimuthTolerance)
+            if (Math.abs(azimuth) <= Math.abs(desiredAzimuth - desiredAzimuthTolerance) || Math.abs(azimuth) <= desiredAzimuth + desiredAzimuthTolerance)
             {
                 tvAppliesAzimuth.setText(getResources().getString(R.string.yes));
                 tvAppliesAzimuth.setTextColor(Color.GREEN);
@@ -51,10 +53,17 @@ public class ActivityManageTriggerDevicePosition extends Activity
                 tvAppliesAzimuth.setText(getResources().getString(R.string.no));
                 tvAppliesAzimuth.setTextColor(Color.RED);
             }
+        }
+        catch(Exception e)
+        {
+            tvAppliesAzimuth.setText("");
+        }
 
+        try
+        {
             desiredPitch = Float.parseFloat(etDesiredPitch.getText().toString());
             desiredPitchTolerance = Float.parseFloat(etDesiredPitchTolerance.getText().toString());
-            if(Math.abs(pitch) <= Math.abs(desiredPitch - desiredPitchTolerance) || Math.abs(pitch) <= desiredPitch + desiredPitchTolerance)
+            if (Math.abs(pitch) <= Math.abs(desiredPitch - desiredPitchTolerance) || Math.abs(pitch) <= desiredPitch + desiredPitchTolerance)
             {
                 tvAppliesPitch.setText(getResources().getString(R.string.yes));
                 tvAppliesPitch.setTextColor(Color.GREEN);
@@ -64,10 +73,17 @@ public class ActivityManageTriggerDevicePosition extends Activity
                 tvAppliesPitch.setText(getResources().getString(R.string.no));
                 tvAppliesPitch.setTextColor(Color.RED);
             }
+        }
+        catch(Exception e)
+        {
+            tvAppliesPitch.setText("");
+        }
 
+        try
+        {
             desiredRoll = Float.parseFloat(etDesiredRoll.getText().toString());
             desiredRollTolerance = Float.parseFloat(etDesiredRollTolerance.getText().toString());
-            if(Math.abs(roll) <= Math.abs(desiredRoll - desiredRollTolerance) || Math.abs(roll) <= desiredRoll + desiredRollTolerance)
+            if (Math.abs(roll) <= Math.abs(desiredRoll - desiredRollTolerance) || Math.abs(roll) <= desiredRoll + desiredRollTolerance)
             {
                 tvAppliesRoll.setText(getResources().getString(R.string.yes));
                 tvAppliesRoll.setTextColor(Color.GREEN);
@@ -78,6 +94,10 @@ public class ActivityManageTriggerDevicePosition extends Activity
                 tvAppliesRoll.setTextColor(Color.RED);
             }
         }
+        catch(Exception e)
+        {
+            tvAppliesRoll.setText("");
+        }
     }
 
     @Override
@@ -103,6 +123,8 @@ public class ActivityManageTriggerDevicePosition extends Activity
         etDesiredRoll = (EditText) findViewById(R.id.etDesiredRoll);
         etDesiredRollTolerance = (EditText) findViewById(R.id.etDesiredRollTolerance);
 
+        chkDevicePositionApplies = (CheckBox)findViewById(R.id.chkDevicePositionApplies);
+
 //        etDesiredAzimuth.setFilters(new InputFilter[]{new InputFilterMinMax(-180, 180)});
 //        etDesiredPitch.setFilters(new InputFilter[]{new InputFilterMinMax(-180, 180)});
 //        etDesiredRoll.setFilters(new InputFilter[]{new InputFilterMinMax(-180, 180)});
@@ -115,6 +137,8 @@ public class ActivityManageTriggerDevicePosition extends Activity
             editMode = true;
             try
             {
+                boolean chkValue = getIntent().getBooleanExtra(ActivityManageRule.intentNameTriggerParameter1, true);
+                chkDevicePositionApplies.setChecked(chkValue);
                 String values[] = getIntent().getStringExtra(vectorFieldName).split(Trigger.triggerParameter2Split);
                 etDesiredAzimuth.setText(values[0]);
                 etDesiredAzimuthTolerance.setText(values[1]);
@@ -159,6 +183,7 @@ public class ActivityManageTriggerDevicePosition extends Activity
                 {
                     // Save
                     Intent returnData = new Intent();
+                    returnData.putExtra(ActivityManageRule.intentNameTriggerParameter1, chkDevicePositionApplies.isChecked());
                     returnData.putExtra(vectorFieldName,
                                         etDesiredAzimuth.getText().toString() + Trigger.triggerParameter2Split +
                                             etDesiredAzimuthTolerance.getText().toString() + Trigger.triggerParameter2Split +

app/src/main/java/com/jens/automation2/ReceiverCoordinator.java

@@ -360,7 +360,7 @@ public class ReceiverCoordinator
             if(!DevicePositionListener.getInstance().isListenerRunning())
             {
                 Miscellaneous.logEvent("i", "DevicePositionListener", "Starting DevicePositionListener because used in a new/changed rule.", 4);
-                if(HeadphoneJackListener.getInstance().haveAllPermission())
+//                if(DevicePositionListener.getInstance().haveAllPermission())
                     DevicePositionListener.getInstance().startListener(AutomationService.getInstance());
             }
         }

app/src/main/java/com/jens/automation2/Trigger.java

@@ -300,7 +300,10 @@ public class Trigger
 		)
 		{
 			Miscellaneous.logEvent("i", "DevicePosition", "Trigger doesn\'t apply. Azimuth outside of tolerance area.", 5);
+			if(getTriggerParameter())
 				return false;
+			else
+				return true;
 		}
 
 		if(
@@ -314,7 +317,10 @@ public class Trigger
 		)
 		{
 			Miscellaneous.logEvent("i", "DevicePosition", "Trigger doesn\'t apply. Pitch outside of tolerance area.", 5);
+			if(getTriggerParameter())
 				return false;
+			else
+				return true;
 		}
 
 		if(
@@ -328,10 +334,16 @@ public class Trigger
 		)
 		{
 			Miscellaneous.logEvent("i", "DevicePosition", "Trigger doesn\'t apply. Roll outside of tolerance area.", 5);
+			if(getTriggerParameter())
 				return false;
+			else
+				return true;
 		}
 
+		if(getTriggerParameter())
 			return true;
+		else
+			return false;
 	}
 
     boolean checkHeadsetPlugged()
@@ -1383,8 +1395,6 @@ public class Trigger
 				break;
 			case bluetoothConnection:
 				String device = Miscellaneous.getAnyContext().getResources().getString(R.string.anyDevice);
-//				if(this.bluetoothDeviceAddress != null)
-//				{
 				if(bluetoothDeviceAddress.equals("<any>"))
 				{
 					device = Miscellaneous.getAnyContext().getResources().getString(R.string.any);
@@ -1401,8 +1411,8 @@ public class Trigger
 					}
 					catch(NullPointerException e)
 					{
-							device = Miscellaneous.getAnyContext().getResources().getString(R.string.invalidDevice) + ": " + device;
+						device = Miscellaneous.getAnyContext().getResources().getString(R.string.invalidDevice) + ": " + this.bluetoothDeviceAddress;
-							Miscellaneous.logEvent("w", "Trigger", Miscellaneous.getAnyContext().getResources().getString(R.string.invalidDevice) + ": " + device, 3);
+						Miscellaneous.logEvent("w", "Trigger", device, 3);
 					}
 				}
 
@@ -1420,7 +1430,6 @@ public class Trigger
 					else
 						returnString.append(String.format(Miscellaneous.getAnyContext().getResources().getString(R.string.bluetoothDeviceOutOfRange), device));
 				}
-//				}
 				break;
 			case headsetPlugged:
 				String type;

app/src/main/java/com/jens/automation2/receivers/DevicePositionListener.java

@@ -100,7 +100,7 @@ public class DevicePositionListener implements SensorEventListener, AutomationLi
             {
                 //unregister the sensor listener
                 sManager.unregisterListener(this);
-                isRunning = true;
+                isRunning = false;
             }
         }
     }
@@ -184,7 +184,7 @@ public class DevicePositionListener implements SensorEventListener, AutomationLi
         {
             //unregister the sensor listener
             sManager.unregisterListener(this);
-            isRunning = true;
+            isRunning = false;
         }
     }
 

app/src/main/java/com/jens/automation2/receivers/HeadphoneJackListener.java

@@ -1,5 +1,6 @@
 package com.jens.automation2.receivers;
 
+import android.Manifest;
 import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
@@ -133,7 +134,7 @@ public class HeadphoneJackListener extends BroadcastReceiver implements Automati
 
 	public static boolean haveAllPermission()
 	{
-		return ActivityPermissions.havePermission("android.permission.READ_PHONE_STATE", Miscellaneous.getAnyContext());
+		return ActivityPermissions.havePermission(Manifest.permission.READ_PHONE_STATE, Miscellaneous.getAnyContext());
 	}
 
 

app/src/main/res/layout/activity_manage_trigger_device_position.xml

@@ -224,6 +224,18 @@
 
         </TableLayout>
 
+        <CheckBox
+            android:id="@+id/chkDevicePositionApplies"
+            android:checked="true"
+            android:text="@string/mustApply"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content" />
+
+        <TextView
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:text="@string/explanationDevicePositionDirection" />
+
         <Button
             android:id="@+id/bSavePositionValues"
             android:text="@string/save"

app/src/main/res/values/strings.xml

@@ -404,8 +404,8 @@
     <string name="detectedActivityWalking">Walking</string>
     <string name="detectedActivityRunning">Running</string>
     <string name="detectedActivityInvalidStatus">Invalid activity</string>
-    <string name="ruleDoesntApplyActivityGivenButTooLowProbability" translatable="false">Rule doesn\'t apply. Detected activity %1$s given, but too low probability (%2$s %%), required %3$s %%.</string>
+    <string name="ruleDoesntApplyActivityGivenButTooLowProbability" translatable="false">Rule %1$s doesn\'t apply. Detected activity %2$s given, but too low probability (%3$s %%), required %4$s %%.</string>
-    <string name="ruleDoesntApplyActivityNotPresent" translatable="false">Rule doesn\'t apply. Required activity %1$s not present.</string>
+    <string name="ruleDoesntApplyActivityNotPresent" translatable="false">Rule %1$s doesn\'t apply. Required activity %2$s not present.</string>
     <string name="selectTypeOfActivity">Select type of activity</string>
     <string name="triggerOnlyAvailableIfPlayServicesInstalled">This trigger is only available if Google Play Services is installed.</string>
     <string name="activityDetectionFrequencyTitle">Activity detection frequency [sec]</string>
@@ -729,4 +729,6 @@
     <string name="turnScreenOnOrOff">Turn screen on or off</string>
     <string name="turnScreenOn">turn screen on</string>
     <string name="turnScreenOff">turn screen off</string>
+    <string name="mustApply">Must apply</string>
+    <string name="explanationDevicePositionDirection">If the checkbox is checked that means the device has to be in the position you specify. If it\'s not checked, any position that does NOT match your criteria will do.</string>
 </resources>

build.gradle

@@ -5,7 +5,7 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:7.0.3'
+        classpath 'com.android.tools.build:gradle:7.0.4'
 
         // NOTE: Do not place your application dependencies here; they belong
         // in the individual module build.gradle files