Browse Source

Remove button order logic from dialog

Signed-off-by: alperozturk <alper_ozturk@proton.me>
alperozturk 1 year ago
parent
commit
24f536ca40

+ 1 - 3
app/src/main/java/com/owncloud/android/ui/activity/UploadFilesActivity.java

@@ -528,9 +528,7 @@ public class UploadFilesActivity extends DrawerActivity implements LocalFileList
             // to the ownCloud folder instead of copying
             String[] args = {getString(R.string.app_name)};
             ConfirmationDialogFragment dialog = ConfirmationDialogFragment.newInstance(
-                R.string.upload_query_move_foreign_files, args, 0, R.string.common_yes, -1,
-                R.string.common_no
-                                                                                      );
+                R.string.upload_query_move_foreign_files, args, 0, R.string.common_yes,  R.string.common_no, -1);
             dialog.setOnConfirmationListener(this);
             dialog.show(getSupportFragmentManager(), QUERY_TO_MOVE_DIALOG_TAG);
         }

+ 5 - 18
app/src/main/java/com/owncloud/android/ui/dialog/ConfirmationDialogFragment.java

@@ -63,7 +63,7 @@ public class ConfirmationDialogFragment extends DialogFragment implements Inject
      * @return Dialog ready to show.
      */
     public static ConfirmationDialogFragment newInstance(int messageResId, String[] messageArguments, int titleResId,
-                                                         int positiveButtonTextId, int neutralButtonTextId, int negativeButtonTextId) {
+                                                         int positiveButtonTextId, int negativeButtonTextId, int neutralButtonTextId) {
         if (messageResId == -1) {
             throw new IllegalStateException("Calling confirmation dialog without message resource");
         }
@@ -156,29 +156,16 @@ public class ConfirmationDialogFragment extends DialogFragment implements Inject
                 dialog.dismiss();
             });
         }
-
-        if (negativeButtonTextId != -1 && neutralButtonTextId == -1) {
+        if (negativeButtonTextId != -1) {
             builder.setNegativeButton(negativeButtonTextId, (dialog, which) -> {
                 if (mListener != null) {
                     mListener.onCancel(getTag());
                 }
                 dialog.dismiss();
             });
-        } else if (neutralButtonTextId != -1 && negativeButtonTextId == -1) {
-            builder.setNegativeButton(neutralButtonTextId, (dialog, which) -> {
-                if (mListener != null) {
-                    mListener.onNeutral(getTag());
-                }
-                dialog.dismiss();
-            });
-        } else {
-            builder.setNegativeButton(negativeButtonTextId, (dialog, which) -> {
-                if (mListener != null) {
-                    mListener.onCancel(getTag());
-                }
-                dialog.dismiss();
-            });
-            builder.setNeutralButton(neutralButtonTextId, (dialog, whichButton) -> {
+        }
+        if (neutralButtonTextId != -1) {
+            builder.setNeutralButton(neutralButtonTextId, (dialog, which) -> {
                 if (mListener != null) {
                     mListener.onNeutral(getTag());
                 }