diff --git a/app/src/main/java/eu/faircode/email/EntityRule.java b/app/src/main/java/eu/faircode/email/EntityRule.java index 1f8d8d7442..3ed104d85a 100644 --- a/app/src/main/java/eu/faircode/email/EntityRule.java +++ b/app/src/main/java/eu/faircode/email/EntityRule.java @@ -690,9 +690,9 @@ public class EntityRule { long aid = jargs.getLong("answer"); boolean answer_subject = jargs.optBoolean("answer_subject", false); boolean original_text = jargs.optBoolean("original_text", true); + boolean attachments = jargs.optBoolean("attachments"); String to = jargs.optString("to"); boolean cc = jargs.optBoolean("cc"); - boolean attachments = jargs.optBoolean("attachments"); boolean isReply = TextUtils.isEmpty(to); diff --git a/app/src/main/java/eu/faircode/email/FragmentRule.java b/app/src/main/java/eu/faircode/email/FragmentRule.java index 00f0eae7e5..75db0cf238 100644 --- a/app/src/main/java/eu/faircode/email/FragmentRule.java +++ b/app/src/main/java/eu/faircode/email/FragmentRule.java @@ -142,10 +142,10 @@ public class FragmentRule extends FragmentBase { private Spinner spAnswer; private CheckBox cbAnswerSubject; private CheckBox cbOriginalText; + private CheckBox cbWithAttachments; private EditText etTo; private ImageButton ibTo; private CheckBox cbCc; - private CheckBox cbWithAttachments; private Button btnTtsSetup; private Button btnTtsData; @@ -294,10 +294,10 @@ public class FragmentRule extends FragmentBase { spAnswer = view.findViewById(R.id.spAnswer); cbAnswerSubject = view.findViewById(R.id.cbAnswerSubject); cbOriginalText = view.findViewById(R.id.cbOriginalText); + cbWithAttachments = view.findViewById(R.id.cbWithAttachments); etTo = view.findViewById(R.id.etTo); ibTo = view.findViewById(R.id.ibTo); cbCc = view.findViewById(R.id.cbCc); - cbWithAttachments = view.findViewById(R.id.cbWithAttachments); btnTtsSetup = view.findViewById(R.id.btnTtsSetup); btnTtsData = view.findViewById(R.id.btnTtsData); @@ -1143,10 +1143,10 @@ public class FragmentRule extends FragmentBase { cbAnswerSubject.setChecked(jaction.optBoolean("answer_subject", false)); cbOriginalText.setChecked(jaction.optBoolean("original_text", true)); + cbWithAttachments.setChecked(jaction.optBoolean("attachments")); etTo.setText(jaction.optString("to")); cbCc.setChecked(jaction.optBoolean("cc")); - cbWithAttachments.setChecked(jaction.optBoolean("attachments")); break; case EntityRule.TYPE_SOUND: @@ -1486,9 +1486,9 @@ public class FragmentRule extends FragmentBase { jaction.put("answer", answer == null || answer.id == null ? -1 : answer.id); jaction.put("answer_subject", cbAnswerSubject.isChecked()); jaction.put("original_text", cbOriginalText.isChecked()); + jaction.put("attachments", cbWithAttachments.isChecked()); jaction.put("to", etTo.getText().toString().trim()); jaction.put("cc", cbCc.isChecked()); - jaction.put("attachments", cbWithAttachments.isChecked()); break; case EntityRule.TYPE_SOUND: diff --git a/app/src/main/res/layout/fragment_rule.xml b/app/src/main/res/layout/fragment_rule.xml index 36194a7476..8d32314dc5 100644 --- a/app/src/main/res/layout/fragment_rule.xml +++ b/app/src/main/res/layout/fragment_rule.xml @@ -859,6 +859,15 @@ app:layout_constraintStart_toStartOf="parent" app:layout_constraintTop_toBottomOf="@id/cbAnswerSubject" /> + + + app:layout_constraintTop_toBottomOf="@id/cbWithAttachments" /> - - + app:layout_constraintTop_toBottomOf="@id/cbCc" />