#3503 Remove Title/Caption From MediaUploadDetail and only use Caption/Description pairs

app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.java

@@ -2,6 +2,7 @@
 
 import androidx.annotation.NonNull;
 
+import okhttp3.logging.HttpLoggingInterceptor.Level;
 import org.wikipedia.dataclient.SharedPreferenceCookieManager;
 import org.wikipedia.dataclient.okhttp.HttpStatusException;
 
@@ -40,7 +41,7 @@ private static OkHttpClient createClient() {
 
     private static HttpLoggingInterceptor getLoggingInterceptor() {
         HttpLoggingInterceptor httpLoggingInterceptor = new HttpLoggingInterceptor()
-                .setLevel(HttpLoggingInterceptor.Level.BASIC);
+                .setLevel(BuildConfig.DEBUG ? Level.BODY: Level.BASIC);
 
         httpLoggingInterceptor.redactHeader("Authorization");
         httpLoggingInterceptor.redactHeader("Cookie");

app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.java

@@ -71,7 +71,7 @@ public HttpLoggingInterceptor provideHttpLoggingInterceptor() {
         HttpLoggingInterceptor httpLoggingInterceptor = new HttpLoggingInterceptor(message -> {
             Timber.tag("OkHttp").v(message);
         });
-        httpLoggingInterceptor.level(BuildConfig.DEBUG ? Level.BODY: HttpLoggingInterceptor.Level.BASIC);
+        httpLoggingInterceptor.level(BuildConfig.DEBUG ? Level.BODY: Level.BASIC);
         return httpLoggingInterceptor;
     }
 

app/src/main/java/fr/free/nrw/commons/upload/UploadMediaDetailAdapter.java

@@ -62,7 +62,7 @@ public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
 
     @Override
     public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
-        holder.init(position);
+        holder.bind(position);
     }
 
     @Override
@@ -102,14 +102,18 @@ public ViewHolder(View itemView) {
             Timber.i("descItemEditText:" + descItemEditText);
         }
 
-        public void init(int position) {
+        public void bind(int position) {
             UploadMediaDetail uploadMediaDetail = uploadMediaDetails.get(position);
             Timber.d("UploadMediaDetail is " + uploadMediaDetail);
             captionItemEditText.setText(uploadMediaDetail.getCaptionText());
             descItemEditText.setText(uploadMediaDetail.getDescriptionText());
 
             captionItemEditText.addTextChangedListener(new AbstractTextWatcher(
-                value -> eventListener.onEvent(value.length() != 0)) );
+                value -> {
+                    if (position == 0) {
+                        eventListener.onPrimaryCaptionTextChange(value.length() != 0);
+                    }
+                }));
 
             if (position == 0) {
                 captionItemEditText.setCompoundDrawablesWithIntrinsicBounds(null, null, getInfoIcon(),
@@ -231,7 +235,7 @@ public interface Callback {
     }
 
     public interface EventListener {
-        void onEvent(Boolean data);
+        void onPrimaryCaptionTextChange(boolean isNotEmpty);
     }
 
     /**

app/src/main/java/fr/free/nrw/commons/upload/UploadModel.java

@@ -119,7 +119,6 @@ private UploadItem createAndAddUploadItem(final UploadableFile uploadableFile,
                 uploadableFile.getMimeType(context), source, imageCoordinates, place, fileCreatedDate,
                 createdTimestampSource);
         if (place != null) {
-            uploadItem.title.setTitleText(place.name);
             uploadItem.getUploadMediaDetails().set(0, new UploadMediaDetail(place));
         }
         if (!items.contains(uploadItem)) {
@@ -207,7 +206,6 @@ public List<UploadItem> getItems() {
     public void updateUploadItem(final int index, final UploadItem uploadItem) {
         final UploadItem uploadItem1 = items.get(index);
         uploadItem1.setMediaDetails(uploadItem.uploadMediaDetails);
-        uploadItem1.setTitle(uploadItem.title);
     }
 
     public void setSelectedDepictions(final List<String> selectedDepictions) {
@@ -232,7 +230,6 @@ public static class UploadItem {
         private final String mimeType;
         private final String source;
         private ImageCoordinates gpsCoords;
-        private Title title;
         private List<UploadMediaDetail> uploadMediaDetails;
         private final Place place;
         private final long createdTimestamp;
@@ -246,8 +243,6 @@ public static class UploadItem {
                 final String createdTimestampSource) {
             this.originalContentUri = originalContentUri;
             this.createdTimestampSource = createdTimestampSource;
-            title = new Title();
-            uploadMediaDetails = Collections.singletonList(new UploadMediaDetail());
             uploadMediaDetails = new ArrayList<>(Arrays.asList(new UploadMediaDetail()));
             this.place = place;
             this.mediaUri = mediaUri;
@@ -278,10 +273,6 @@ public long getCreatedTimestamp() {
             return createdTimestamp;
         }
 
-        public Title getTitle() {
-            return title;
-        }
-
         public Uri getMediaUri() {
             return mediaUri;
         }
@@ -298,11 +289,6 @@ public Place getPlace() {
             return place;
         }
 
-        public void setTitle(final Title title) {
-            this.title = title;
-        }
-
-
         public void setMediaDetails(final List<UploadMediaDetail> uploadMediaDetails) {
             this.uploadMediaDetails = uploadMediaDetails;
         }

app/src/main/java/fr/free/nrw/commons/upload/categories/CategoriesPresenter.java

@@ -1,15 +1,9 @@
 package fr.free.nrw.commons.upload.categories;
 
-import android.text.TextUtils;
-
-import java.lang.reflect.Proxy;
-import java.util.ArrayList;
-import java.util.List;
-
-import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
+import static fr.free.nrw.commons.di.CommonsApplicationModule.IO_THREAD;
+import static fr.free.nrw.commons.di.CommonsApplicationModule.MAIN_THREAD;
 
+import android.text.TextUtils;
 import fr.free.nrw.commons.R;
 import fr.free.nrw.commons.category.CategoryItem;
 import fr.free.nrw.commons.repository.UploadRepository;
@@ -18,11 +12,14 @@
 import io.reactivex.Scheduler;
 import io.reactivex.disposables.CompositeDisposable;
 import io.reactivex.disposables.Disposable;
+import java.lang.reflect.Proxy;
+import java.util.ArrayList;
+import java.util.List;
+import javax.inject.Inject;
+import javax.inject.Named;
+import javax.inject.Singleton;
 import timber.log.Timber;
 
-import static fr.free.nrw.commons.di.CommonsApplicationModule.IO_THREAD;
-import static fr.free.nrw.commons.di.CommonsApplicationModule.MAIN_THREAD;
-
 /**
  * The presenter class for UploadCategoriesFragment
  */
@@ -86,9 +83,10 @@ public void searchForCategories(String query) {
                 )
                 .filter(categoryItem -> !repository.containsYear(categoryItem.getName()))
                 .distinct();
-                if(!TextUtils.isEmpty(query)) {
-                distinctCategoriesObservable=distinctCategoriesObservable.sorted(repository.sortBySimilarity(query));
-                }
+
+        if(!TextUtils.isEmpty(query)) {
+            distinctCategoriesObservable=distinctCategoriesObservable.sorted(repository.sortBySimilarity(query));
+        }
         Disposable searchCategoriesDisposable = distinctCategoriesObservable
                 .observeOn(mainThreadScheduler)
                 .subscribe(
@@ -114,8 +112,9 @@ public void searchForCategories(String query) {
     private List<String> getImageTitleList() {
         List<String> titleList = new ArrayList<>();
         for (UploadItem item : repository.getUploads()) {
-            if (item.getTitle().isSet()) {
-                titleList.add(item.getTitle().toString());
+            final String captionText = item.getUploadMediaDetails().get(0).getCaptionText();
+            if (!TextUtils.isEmpty(captionText)) {
+                titleList.add(captionText);
             }
         }
         return titleList;

app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaDetailFragment.java

@@ -3,15 +3,10 @@
 import static fr.free.nrw.commons.utils.ImageUtils.getErrorMessageForResult;
 
 import android.annotation.SuppressLint;
-import android.content.Context;
 import android.os.Bundle;
-import android.text.TextUtils;
-import android.util.DisplayMetrics;
 import android.view.LayoutInflater;
-import android.view.MotionEvent;
 import android.view.View;
 import android.view.ViewGroup;
-import android.widget.EditText;
 import android.widget.LinearLayout;
 import android.widget.TextView;
 import androidx.annotation.NonNull;
@@ -24,7 +19,6 @@
 import butterknife.ButterKnife;
 import butterknife.OnClick;
 import com.github.chrisbanes.photoview.PhotoView;
-import com.jakewharton.rxbinding2.widget.RxTextView;
 import fr.free.nrw.commons.R;
 import fr.free.nrw.commons.Utils;
 import fr.free.nrw.commons.filepicker.UploadableFile;
@@ -34,7 +28,6 @@
 import fr.free.nrw.commons.settings.Prefs;
 import fr.free.nrw.commons.upload.ImageCoordinates;
 import fr.free.nrw.commons.upload.SimilarImageDialogFragment;
-import fr.free.nrw.commons.upload.Title;
 import fr.free.nrw.commons.upload.UploadBaseFragment;
 import fr.free.nrw.commons.upload.UploadMediaDetail;
 import fr.free.nrw.commons.upload.UploadMediaDetailAdapter;
@@ -43,7 +36,6 @@
 import fr.free.nrw.commons.utils.DialogUtil;
 import fr.free.nrw.commons.utils.ImageUtils;
 import fr.free.nrw.commons.utils.ViewUtil;
-import io.reactivex.disposables.Disposable;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
@@ -66,8 +58,6 @@ public class UploadMediaDetailFragment extends UploadBaseFragment implements
     AppCompatImageButton ibExpandCollapse;
     @BindView(R.id.ll_container_media_detail)
     LinearLayout llContainerMediaDetail;
-    @BindView(R.id.et_title)
-    EditText etTitle;
     @BindView(R.id.rv_descriptions)
     RecyclerView rvDescriptions;
     @BindView(R.id.backgroundImage)
@@ -94,7 +84,6 @@ public class UploadMediaDetailFragment extends UploadBaseFragment implements
     private String source;
     private Place place;
 
-    private Title title;
     private boolean isExpanded = true;
 
     private UploadMediaDetailFragmentCallback callback;
@@ -131,24 +120,8 @@ public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceStat
     private void init() {
         tvTitle.setText(getString(R.string.step_count, callback.getIndexInViewFlipper(this) + 1,
                 callback.getTotalNumberOfSteps()));
-        title = new Title();
         initRecyclerView();
         initPresenter();
-        Disposable disposable = RxTextView.textChanges(etTitle)
-                .subscribe(text -> {
-                    if (!TextUtils.isEmpty(text)) {
-                        btnNext.setEnabled(true);
-                        btnNext.setClickable(true);
-                        btnNext.setAlpha(1.0f);
-                        title.setTitleText(text.toString());
-                        uploadItem.setTitle(title);
-                    } else {
-                        btnNext.setAlpha(0.5f);
-                        btnNext.setEnabled(false);
-                        btnNext.setClickable(false);
-                    }
-                });
-        compositeDisposable.add(disposable);
         presenter.receiveImage(uploadableFile, source, place);
 
         if (callback.getIndexInViewFlipper(this) == 0) {
@@ -168,36 +141,6 @@ private void init() {
 
         attachImageViewScaleChangeListener();
 
-        addEtTitleTouchListener();
-    }
-
-    /**
-     * Handles the drawable click listener for Edit Text
-     */
-    private void addEtTitleTouchListener() {
-        etTitle.setOnTouchListener((v, event) -> {
-            //2 is for drawable right
-            float twelveDpInPixels = convertDpToPixel(12, getContext());
-            if (event.getAction() == MotionEvent.ACTION_UP && etTitle.getCompoundDrawables() != null
-                    && etTitle.getCompoundDrawables().length > 2 && etTitle
-                    .getCompoundDrawables()[2].getBounds()
-                    .contains((int) (etTitle.getWidth() - (event.getX() + twelveDpInPixels)),
-                            (int) (event.getY() - twelveDpInPixels))) {
-                showInfoAlert(R.string.media_detail_title, R.string.title_info);
-                return true;
-            }
-            return false;
-        });
-    }
-
-    /**
-     * converts dp to pixel
-     * @param dp
-     * @param context
-     * @return
-     */
-    private float convertDpToPixel(float dp, Context context) {
-        return dp * ((float) context.getResources().getDisplayMetrics().densityDpi / DisplayMetrics.DENSITY_DEFAULT);
     }
 
     /**
@@ -224,7 +167,7 @@ private void initPresenter() {
     private void initRecyclerView() {
         uploadMediaDetailAdapter = new UploadMediaDetailAdapter(defaultKvStore.getString(Prefs.KEY_LANGUAGE_VALUE, ""));
         uploadMediaDetailAdapter.setCallback(this::showInfoAlert);
-        uploadMediaDetailAdapter.setEventListener(this::onEvent);
+        uploadMediaDetailAdapter.setEventListener(this);
         rvDescriptions.setLayoutManager(new LinearLayoutManager(getContext()));
         rvDescriptions.setAdapter(uploadMediaDetailAdapter);
     }
@@ -301,8 +244,7 @@ public void onNearbyPlaceFound(UploadItem uploadItem, Place place) {
                         getString(R.string.upload_nearby_place_found_description),
                         place.getName()),
                 () -> {
-                    etTitle.setText(place.getName());
-                    descriptions = new ArrayList<>(Arrays.asList(new UploadMediaDetail()));
+                    descriptions = new ArrayList<>(Arrays.asList(new UploadMediaDetail(place)));
                     setDescriptionsInAdapter(descriptions);
                 },
                 () -> {
@@ -368,8 +310,7 @@ public void showBadImagePopup(Integer errorCode) {
     }
 
     @Override
-    public void setTitleAndDescription(String title, List<UploadMediaDetail> uploadMediaDetails) {
-        etTitle.setText(title);
+    public void setCaptionsAndDescriptions(List<UploadMediaDetail> uploadMediaDetails) {
         setDescriptionsInAdapter(uploadMediaDetails);
     }
 
@@ -405,10 +346,10 @@ private void expandCollapseLlMediaDetail(boolean shouldExpand){
     }
 
     @Override
-    public void onEvent(Boolean data) {
-        btnNext.setEnabled(data);
-        btnNext.setClickable(data);
-        btnNext.setAlpha(data ? 1.0f: 0.5f);
+    public void onPrimaryCaptionTextChange(boolean isNotEmpty) {
+        btnNext.setEnabled(isNotEmpty);
+        btnNext.setClickable(isNotEmpty);
+        btnNext.setAlpha(isNotEmpty ? 1.0f: 0.5f);
     }
 
 

app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaDetailsContract.java

@@ -36,7 +36,7 @@ interface View extends SimilarImageInterface {
 
         void showMapWithImageCoordinates(boolean shouldShow);
 
-        void setTitleAndDescription(String title, List<UploadMediaDetail> uploadMediaDetails);
+        void setCaptionsAndDescriptions(List<UploadMediaDetail> uploadMediaDetails);
     }
 
     interface UserActionListener extends BasePresenter<View> {

app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaPresenter.java

@@ -151,7 +151,7 @@ public void setUploadItem(int index, UploadItem uploadItem) {
     public void fetchPreviousTitleAndDescription(int indexInViewFlipper) {
         UploadItem previousUploadItem = repository.getPreviousUploadItem(indexInViewFlipper);
         if (null != previousUploadItem) {
-            view.setTitleAndDescription(previousUploadItem.getTitle().getTitleText(), previousUploadItem.getUploadMediaDetails());
+            view.setCaptionsAndDescriptions(previousUploadItem.getUploadMediaDetails());
         } else {
             view.showMessage(R.string.previous_image_title_description_not_found, R.color.color_error);
         }