Skip to content

Commit

Permalink
moved all static references to EditPostBaseActivity
Browse files Browse the repository at this point in the history
  • Loading branch information
mzorz committed Dec 24, 2018
1 parent a49bc70 commit 74765be
Show file tree
Hide file tree
Showing 13 changed files with 39 additions and 40 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@
import org.wordpress.android.ui.plugins.PluginDetailActivity;
import org.wordpress.android.ui.plugins.PluginUtils;
import org.wordpress.android.ui.posts.EditPostActivity;
import org.wordpress.android.ui.posts.EditPostBaseActivity;
import org.wordpress.android.ui.posts.PostPreviewActivity;
import org.wordpress.android.ui.posts.PostsListActivity;
import org.wordpress.android.ui.prefs.AccountSettingsActivity;
Expand Down Expand Up @@ -571,14 +572,14 @@ public static void viewPostPreviewForResult(Activity activity, SiteModel site, P
}

Intent intent = new Intent(activity, PostPreviewActivity.class);
intent.putExtra(EditPostActivity.EXTRA_POST_LOCAL_ID, post.getId());
intent.putExtra(EditPostBaseActivity.EXTRA_POST_LOCAL_ID, post.getId());
intent.putExtra(WordPress.SITE, site);
activity.startActivityForResult(intent, RequestCodes.PREVIEW_POST);
}

public static void viewPagePreview(@NonNull Fragment fragment, @NonNull PageModel page) {
Intent intent = new Intent(fragment.getContext(), PostPreviewActivity.class);
intent.putExtra(EditPostActivity.EXTRA_POST_LOCAL_ID, page.getPageId());
intent.putExtra(EditPostBaseActivity.EXTRA_POST_LOCAL_ID, page.getPageId());
intent.putExtra(WordPress.SITE, page.getSite());
fragment.startActivity(intent);
}
Expand All @@ -590,8 +591,8 @@ public static void addNewPostForResult(Activity activity, SiteModel site, boolea

Intent intent = new Intent(activity, EditPostActivity.class);
intent.putExtra(WordPress.SITE, site);
intent.putExtra(EditPostActivity.EXTRA_IS_PAGE, false);
intent.putExtra(EditPostActivity.EXTRA_IS_PROMO, isPromo);
intent.putExtra(EditPostBaseActivity.EXTRA_IS_PAGE, false);
intent.putExtra(EditPostBaseActivity.EXTRA_IS_PROMO, isPromo);
activity.startActivityForResult(intent, RequestCodes.EDIT_POST);
}

Expand All @@ -605,22 +606,22 @@ public static void editPostOrPageForResult(Activity activity, SiteModel site, Po
// PostModel objects can be quite large, since content field is not size restricted,
// in order to avoid issues like TransactionTooLargeException it's better to pass the id of the post.
// However, we still want to keep passing the SiteModel to avoid confusion around local & remote ids.
intent.putExtra(EditPostActivity.EXTRA_POST_LOCAL_ID, post.getId());
intent.putExtra(EditPostBaseActivity.EXTRA_POST_LOCAL_ID, post.getId());
activity.startActivityForResult(intent, RequestCodes.EDIT_POST);
}

public static void editPageForResult(@NonNull Fragment fragment, @NonNull PageModel page) {
Intent intent = new Intent(fragment.getContext(), EditPostActivity.class);
intent.putExtra(WordPress.SITE, page.getSite());
intent.putExtra(EditPostActivity.EXTRA_POST_LOCAL_ID, page.getPageId());
intent.putExtra(EditPostBaseActivity.EXTRA_POST_LOCAL_ID, page.getPageId());
fragment.startActivityForResult(intent, RequestCodes.EDIT_POST);
}

public static void addNewPageForResult(@NonNull Fragment fragment, @NonNull SiteModel site) {
Intent intent = new Intent(fragment.getContext(), EditPostActivity.class);
intent.putExtra(WordPress.SITE, site);
intent.putExtra(EditPostActivity.EXTRA_IS_PAGE, true);
intent.putExtra(EditPostActivity.EXTRA_IS_PROMO, false);
intent.putExtra(EditPostBaseActivity.EXTRA_IS_PAGE, true);
intent.putExtra(EditPostBaseActivity.EXTRA_IS_PROMO, false);
fragment.startActivityForResult(intent, RequestCodes.EDIT_POST);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
import org.wordpress.android.fluxc.store.SiteStore;
import org.wordpress.android.fluxc.tools.FluxCImageLoader;
import org.wordpress.android.ui.posts.EditPostActivity;
import org.wordpress.android.ui.posts.EditPostBaseActivity;
import org.wordpress.android.util.LocaleManager;
import org.wordpress.android.util.SiteUtils;
import org.wordpress.android.util.ToastUtils;
Expand Down Expand Up @@ -140,8 +141,8 @@ public void onClick(DialogInterface dialog, int which) {
shortcutIntent.setAction(Intent.ACTION_MAIN);
shortcutIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
shortcutIntent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
shortcutIntent.putExtra(EditPostActivity.EXTRA_QUICKPRESS_BLOG_ID, siteIds[position]);
shortcutIntent.putExtra(EditPostActivity.EXTRA_IS_QUICKPRESS, true);
shortcutIntent.putExtra(EditPostBaseActivity.EXTRA_QUICKPRESS_BLOG_ID, siteIds[position]);
shortcutIntent.putExtra(EditPostBaseActivity.EXTRA_IS_QUICKPRESS, true);

String shortcutName = quickPressShortcutName.getText().toString();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
import org.wordpress.android.ui.notifications.utils.PendingDraftsNotificationsUtils;
import org.wordpress.android.ui.posts.BasicFragmentDialog.BasicDialogNegativeClickInterface;
import org.wordpress.android.ui.posts.BasicFragmentDialog.BasicDialogPositiveClickInterface;
import org.wordpress.android.ui.posts.EditPostActivity;
import org.wordpress.android.ui.posts.EditPostBaseActivity;
import org.wordpress.android.ui.posts.PromoDialog;
import org.wordpress.android.ui.posts.PromoDialog.PromoDialogClickInterface;
import org.wordpress.android.ui.prefs.AppPrefs;
Expand Down Expand Up @@ -686,7 +686,7 @@ public void onActivityResult(int requestCode, int resultCode, Intent data) {
if (resultCode != Activity.RESULT_OK || data == null || isFinishing()) {
return;
}
int localId = data.getIntExtra(EditPostActivity.EXTRA_POST_LOCAL_ID, 0);
int localId = data.getIntExtra(EditPostBaseActivity.EXTRA_POST_LOCAL_ID, 0);
final SiteModel site = getSelectedSite();
final PostModel post = mPostStore.getPostByLocalPostId(localId);
if (site != null && post != null) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ import org.wordpress.android.ui.RequestCodes
import org.wordpress.android.ui.WPWebViewActivity
import org.wordpress.android.ui.pages.PageItem.Page
import org.wordpress.android.ui.posts.BasicFragmentDialog
import org.wordpress.android.ui.posts.EditPostActivity
import org.wordpress.android.ui.posts.EditPostBaseActivity
import org.wordpress.android.ui.posts.GutenbergWarningFragmentDialog.GutenbergWarningDialogClickInterface
import org.wordpress.android.ui.posts.PostUtils
import org.wordpress.android.ui.prefs.AppPrefs
Expand Down Expand Up @@ -90,7 +90,7 @@ class PagesFragment : Fragment(), GutenbergWarningDialogClickInterface {

override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
if (requestCode == RequestCodes.EDIT_POST && resultCode == Activity.RESULT_OK && data != null) {
val pageId = data.getLongExtra(EditPostActivity.EXTRA_POST_REMOTE_ID, -1)
val pageId = data.getLongExtra(EditPostBaseActivity.EXTRA_POST_REMOTE_ID, -1)
if (pageId != -1L) {
onPageEditFinished(pageId)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -207,11 +207,6 @@ public class EditPostActivity extends EditPostBaseActivity implements

private AztecImageLoader mAztecImageLoader;

enum AddExistingdMediaSource {
WP_MEDIA_LIBRARY,
STOCK_PHOTO_LIBRARY
}

private Handler mHandler;
private int mDebounceCounter = 0;
private boolean mShowAztecEditor;
Expand Down Expand Up @@ -2082,7 +2077,7 @@ private int getMaximumThumbnailWidthForEditor() {
return mMaxThumbWidth;
}

private boolean addExistingMediaToEditor(@NonNull AddExistingdMediaSource source, long mediaId) {
private boolean addExistingMediaToEditor(@NonNull AddExistingMediaSource source, long mediaId) {
MediaModel media = mMediaStore.getSiteMediaWithId(mSite, mediaId);
if (media == null) {
AppLog.w(T.MEDIA, "Cannot add null media to post");
Expand Down Expand Up @@ -2270,7 +2265,7 @@ private void prepareMediaPost() {
long[] idsArray = getIntent().getLongArrayExtra(NEW_MEDIA_POST_EXTRA_IDS);
ArrayList<Long> idsList = ListUtils.fromLongArray(idsArray);
for (Long id : idsList) {
addExistingMediaToEditor(AddExistingdMediaSource.WP_MEDIA_LIBRARY, id);
addExistingMediaToEditor(AddExistingMediaSource.WP_MEDIA_LIBRARY, id);
}
savePostAsync(null);
}
Expand Down Expand Up @@ -2471,7 +2466,7 @@ private void trackAddMediaFromDeviceEvents(boolean isNew, boolean isVideo, Uri u
* @param source where the media is being added from
* @param media media being added
*/
private void trackAddMediaEvent(@NonNull AddExistingdMediaSource source, @NonNull MediaModel media) {
private void trackAddMediaEvent(@NonNull AddExistingMediaSource source, @NonNull MediaModel media) {
switch (source) {
case WP_MEDIA_LIBRARY:
if (media.isVideo()) {
Expand Down Expand Up @@ -2787,7 +2782,7 @@ public void done() {
long[] mediaIds =
data.getLongArrayExtra(StockMediaPickerActivity.KEY_UPLOADED_MEDIA_IDS);
for (long id : mediaIds) {
addExistingMediaToEditor(AddExistingdMediaSource.STOCK_PHOTO_LIBRARY, id);
addExistingMediaToEditor(AddExistingMediaSource.STOCK_PHOTO_LIBRARY, id);
}
savePostAsync(null);
}
Expand Down Expand Up @@ -2909,7 +2904,7 @@ private void handleMediaPickerResult(Intent data) {
showInsertMediaDialog(ids);
} else {
for (Long id : ids) {
addExistingMediaToEditor(AddExistingdMediaSource.WP_MEDIA_LIBRARY, id);
addExistingMediaToEditor(AddExistingMediaSource.WP_MEDIA_LIBRARY, id);
}
savePostAsync(null);
}
Expand All @@ -2932,7 +2927,7 @@ public void onCompleted(@NonNull InsertMediaDialog dialog) {
break;
case INDIVIDUALLY:
for (Long id : mediaIds) {
addExistingMediaToEditor(AddExistingdMediaSource.WP_MEDIA_LIBRARY, id);
addExistingMediaToEditor(AddExistingMediaSource.WP_MEDIA_LIBRARY, id);
}
savePostAsync(null);
break;
Expand Down Expand Up @@ -3374,7 +3369,7 @@ public void onEditorFragmentInitialized() {
shouldFinishInit = false;
mMediaInsertedOnCreation = true;
for (MediaModel media : mediaList) {
addExistingMediaToEditor(AddExistingdMediaSource.WP_MEDIA_LIBRARY, media.getMediaId());
addExistingMediaToEditor(AddExistingMediaSource.WP_MEDIA_LIBRARY, media.getMediaId());
}
savePostAsync(new AfterSavePostListener() {
@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public abstract class EditPostBaseActivity extends AppCompatActivity {
protected static final int CHANGE_SAVE_DELAY = 500;
public static final int MAX_UNSAVED_POSTS = 50;

protected enum AddExistingdMediaSource {
protected enum AddExistingMediaSource {
WP_MEDIA_LIBRARY,
STOCK_PHOTO_LIBRARY
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@
import javax.inject.Inject;

import static android.app.Activity.RESULT_OK;
import static org.wordpress.android.ui.posts.EditPostActivity.EXTRA_POST_LOCAL_ID;
import static org.wordpress.android.ui.posts.EditPostBaseActivity.EXTRA_POST_LOCAL_ID;
import static org.wordpress.android.ui.posts.SelectCategoriesActivity.KEY_SELECTED_CATEGORY_IDS;

public class EditPostSettingsFragment extends Fragment {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@

import de.greenrobot.event.EventBus;

import static org.wordpress.android.ui.posts.EditPostActivity.EXTRA_POST_LOCAL_ID;
import static org.wordpress.android.ui.posts.EditPostBaseActivity.EXTRA_POST_LOCAL_ID;

public class PostPreviewActivity extends AppCompatActivity implements
BasicFragmentDialog.BasicDialogNegativeClickInterface,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@

import javax.inject.Inject;

import static org.wordpress.android.ui.posts.EditPostActivity.EXTRA_POST_LOCAL_ID;
import static org.wordpress.android.ui.posts.EditPostBaseActivity.EXTRA_POST_LOCAL_ID;

public class PostPreviewFragment extends Fragment {
private SiteModel mSite;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@

import javax.inject.Inject;

import static org.wordpress.android.ui.posts.EditPostActivity.EXTRA_POST_LOCAL_ID;
import static org.wordpress.android.ui.posts.EditPostBaseActivity.EXTRA_POST_LOCAL_ID;
import static org.wordpress.android.util.WPSwipeToRefreshHelper.buildSwipeToRefreshHelper;

public class SelectCategoriesActivity extends AppCompatActivity {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
import org.wordpress.android.ui.notifications.ShareAndDismissNotificationReceiver;
import org.wordpress.android.ui.pages.PagesActivity;
import org.wordpress.android.ui.posts.EditPostActivity;
import org.wordpress.android.ui.posts.EditPostBaseActivity;
import org.wordpress.android.ui.posts.PostUtils;
import org.wordpress.android.ui.posts.PostsListActivity;
import org.wordpress.android.ui.prefs.AppPrefs;
Expand Down Expand Up @@ -411,8 +412,8 @@ void updateNotificationSuccessForMedia(@NonNull List<MediaModel> mediaList, @Non
writePostIntent.addFlags(Intent.FLAG_ACTIVITY_SINGLE_TOP);
writePostIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
writePostIntent.putExtra(WordPress.SITE, site);
writePostIntent.putExtra(EditPostActivity.EXTRA_IS_PAGE, false);
writePostIntent.putExtra(EditPostActivity.EXTRA_INSERT_MEDIA, mediaToIncludeInPost);
writePostIntent.putExtra(EditPostBaseActivity.EXTRA_IS_PAGE, false);
writePostIntent.putExtra(EditPostBaseActivity.EXTRA_INSERT_MEDIA, mediaToIncludeInPost);
writePostIntent.setAction(String.valueOf(notificationId));

PendingIntent actionPendingIntent =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
import org.wordpress.android.fluxc.store.UploadStore.UploadError;
import org.wordpress.android.ui.ActivityLauncher;
import org.wordpress.android.ui.posts.EditPostActivity;
import org.wordpress.android.ui.posts.EditPostBaseActivity;
import org.wordpress.android.ui.posts.PostUtils;
import org.wordpress.android.ui.prefs.AppPrefs;
import org.wordpress.android.util.AccessibilityUtils;
Expand Down Expand Up @@ -101,21 +102,21 @@ public static void handleEditPostResultSnackbars(@NonNull final Activity activit
@NonNull final PostModel post,
@NonNull final SiteModel site,
View.OnClickListener publishPostListener) {
boolean hasChanges = data.getBooleanExtra(EditPostActivity.EXTRA_HAS_CHANGES, false);
boolean hasChanges = data.getBooleanExtra(EditPostBaseActivity.EXTRA_HAS_CHANGES, false);
if (!hasChanges) {
// if there are no changes, we don't need to do anything
return;
}

boolean savedLocally = data.getBooleanExtra(EditPostActivity.EXTRA_SAVED_AS_LOCAL_DRAFT, false);
boolean savedLocally = data.getBooleanExtra(EditPostBaseActivity.EXTRA_SAVED_AS_LOCAL_DRAFT, false);
if (savedLocally && !NetworkUtils.isNetworkAvailable(activity)) {
// The network is not available, we can't do anything
ToastUtils.showToast(activity, R.string.error_publish_no_network,
ToastUtils.Duration.SHORT);
return;
}

boolean hasFailedMedia = data.getBooleanExtra(EditPostActivity.EXTRA_HAS_FAILED_MEDIA, false);
boolean hasFailedMedia = data.getBooleanExtra(EditPostBaseActivity.EXTRA_HAS_FAILED_MEDIA, false);
if (hasFailedMedia) {
showSnackbar(snackbarAttachView, R.string.editor_post_saved_locally_failed_media, R.string.button_edit,
new View.OnClickListener() {
Expand Down Expand Up @@ -395,8 +396,8 @@ public void onClick(View view) {
writePostIntent.addFlags(Intent.FLAG_ACTIVITY_SINGLE_TOP);
writePostIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
writePostIntent.putExtra(WordPress.SITE, site);
writePostIntent.putExtra(EditPostActivity.EXTRA_IS_PAGE, false);
writePostIntent.putExtra(EditPostActivity.EXTRA_INSERT_MEDIA, mediaListToInsertInPost);
writePostIntent.putExtra(EditPostBaseActivity.EXTRA_IS_PAGE, false);
writePostIntent.putExtra(EditPostBaseActivity.EXTRA_INSERT_MEDIA, mediaListToInsertInPost);
activity.startActivity(writePostIntent);
}
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ import org.wordpress.android.fluxc.store.PostStore.RemotePostPayload
import org.wordpress.android.fluxc.store.UploadStore
import org.wordpress.android.ui.notifications.utils.PendingDraftsNotificationsUtils
import org.wordpress.android.ui.pages.SnackbarMessageHolder
import org.wordpress.android.ui.posts.EditPostActivity
import org.wordpress.android.ui.posts.EditPostBaseActivity
import org.wordpress.android.ui.posts.PostAdapterItem
import org.wordpress.android.ui.posts.PostAdapterItemData
import org.wordpress.android.ui.posts.PostAdapterItemUploadStatus
Expand Down Expand Up @@ -244,7 +244,7 @@ class PostListViewModel @Inject constructor(
}

fun handleEditPostResult(data: Intent?) {
val localPostId = data?.getIntExtra(EditPostActivity.EXTRA_POST_LOCAL_ID, 0)
val localPostId = data?.getIntExtra(EditPostBaseActivity.EXTRA_POST_LOCAL_ID, 0)
if (localPostId == null || localPostId == 0) {
return
}
Expand Down

0 comments on commit 74765be

Please sign in to comment.