diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2014-02-24 22:18:28 +0100 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2014-02-24 22:18:28 +0100 |
| commit | 285f22f28fed47062b3db2c0294c4c4f13571458 (patch) | |
| tree | 410c9a07e2ea6f788943d594311e9a207f9c5fea /main/src/cgeo/geocaching/TrackableActivity.java | |
| parent | 68f3ab72b0b189d7c0a847f13744696295e9ae5f (diff) | |
| download | cgeo-285f22f28fed47062b3db2c0294c4c4f13571458.zip cgeo-285f22f28fed47062b3db2c0294c4c4f13571458.tar.gz cgeo-285f22f28fed47062b3db2c0294c4c4f13571458.tar.bz2 | |
refactoring: do not duplicate code
Diffstat (limited to 'main/src/cgeo/geocaching/TrackableActivity.java')
| -rw-r--r-- | main/src/cgeo/geocaching/TrackableActivity.java | 25 |
1 files changed, 1 insertions, 24 deletions
diff --git a/main/src/cgeo/geocaching/TrackableActivity.java b/main/src/cgeo/geocaching/TrackableActivity.java index 90b042e..948e668 100644 --- a/main/src/cgeo/geocaching/TrackableActivity.java +++ b/main/src/cgeo/geocaching/TrackableActivity.java @@ -18,10 +18,8 @@ import cgeo.geocaching.ui.Formatter; import cgeo.geocaching.ui.UserActionsClickListener; import cgeo.geocaching.ui.UserNameClickListener; import cgeo.geocaching.ui.logs.TrackableLogsViewCreator; -import cgeo.geocaching.utils.ClipboardUtils; import cgeo.geocaching.utils.HtmlUtils; import cgeo.geocaching.utils.Log; -import cgeo.geocaching.utils.TranslationUtils; import cgeo.geocaching.utils.UnknownTagsHandler; import org.apache.commons.lang3.StringUtils; @@ -223,28 +221,7 @@ public class TrackableActivity extends AbstractViewPagerActivity<TrackableActivi @Override public boolean onContextItemSelected(MenuItem item) { - switch (item.getItemId()) { - // detail fields - case R.id.menu_copy: - ClipboardUtils.copyToClipboard(clickedItemText); - showToast(res.getString(R.string.clipboard_copy_ok)); - return true; - case R.id.menu_translate_to_sys_lang: - TranslationUtils.startActivityTranslate(this, Locale.getDefault().getLanguage(), HtmlUtils.extractText(clickedItemText)); - return true; - case R.id.menu_translate_to_english: - TranslationUtils.startActivityTranslate(this, Locale.ENGLISH.getLanguage(), HtmlUtils.extractText(clickedItemText)); - return true; - case R.id.menu_cache_share_field: - final Intent intent = new Intent(Intent.ACTION_SEND); - intent.setType("text/plain"); - intent.putExtra(Intent.EXTRA_TEXT, clickedItemText.toString()); - startActivity(Intent.createChooser(intent, res.getText(R.string.cache_share_field))); - return true; - default: - break; - } - return onOptionsItemSelected(item); + return onClipboardItemSelected(item, clickedItemText) || onOptionsItemSelected(item); } @Override |
