From 51bf5c21952b99d068238ce462b5ed0467f6225b Mon Sep 17 00:00:00 2001
From: Jacek Jendrzej <crashdvb@googlemail.com>
Date: Fri, 29 Jul 2016 15:34:41 +0200
Subject: [PATCH] src/gui/moviebrowser.cpp use one key for delete cover or add
 tmdb cover

---
 src/gui/moviebrowser.cpp | 72 +++++++++++++++-------------------------
 1 file changed, 27 insertions(+), 45 deletions(-)

diff --git a/src/gui/moviebrowser.cpp b/src/gui/moviebrowser.cpp
index efa8b0f28..862d68763 100644
--- a/src/gui/moviebrowser.cpp
+++ b/src/gui/moviebrowser.cpp
@@ -1849,11 +1849,33 @@ bool CMovieBrowser::onButtonPressMainFrame(neutrino_msg_t msg)
 	else if (msg == (neutrino_msg_t) g_settings.mbkey_cover)
 	{
 		if (m_movieSelectionHandler != NULL) {
-			if (ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_MOVIEBROWSER_DELETE_SCREENSHOT, CMessageBox::mbrNo, CMessageBox:: mbYes | CMessageBox::mbNo) == CMessageBox::mbrYes) {
-				std::string fname = getScreenshotName(m_movieSelectionHandler->file.Name, S_ISDIR(m_movieSelectionHandler->file.Mode));
-				if (!fname.empty())
-					unlink(fname.c_str());
-				refresh();
+			std::string fname = getScreenshotName(m_movieSelectionHandler->file.Name, S_ISDIR(m_movieSelectionHandler->file.Mode));
+			if (!fname.empty()){
+				//delete Cover
+				if (ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_MOVIEBROWSER_DELETE_SCREENSHOT, CMessageBox::mbrNo, CMessageBox:: mbYes | CMessageBox::mbNo) == CMessageBox::mbrYes) {
+						unlink(fname.c_str());
+					refresh();
+				}
+			}else {
+				//add TMDB Cover
+				fname = m_movieSelectionHandler->file.Name.c_str();
+				int ext_pos = 0;
+				ext_pos = fname.rfind('.');
+				if( ext_pos > 0) {
+					std::string extension;
+					extension = fname.substr(ext_pos + 1, fname.length() - ext_pos);
+					extension = "." + extension;
+					strReplace(fname, extension.c_str(), ".jpg");
+					printf("TMDB: %s : %s\n",m_movieSelectionHandler->file.Name.c_str(),fname.c_str());
+					cTmdb* tmdb = new cTmdb(m_movieSelectionHandler->epgTitle);
+					if ((tmdb->getResults() > 0) && (tmdb->hasCover())) {
+						if (!fname.empty())
+							if (tmdb->getSmallCover(fname))
+								refresh();
+					}
+					if (tmdb)
+						delete tmdb;
+				}
 			}
 		}
 	}
@@ -2006,46 +2028,6 @@ bool CMovieBrowser::onButtonPressMainFrame(neutrino_msg_t msg)
 			smsInput.resetOldKey();
 		}
 	}
-	else if (msg == CRCInput::RC_favorites)
-	{
-		if (m_movieSelectionHandler != NULL) {
-#if 0
-			if (ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_MOVIEBROWSER_DELETE_SCREENSHOT, CMessageBox::mbrNo, CMessageBox:: mbYes | CMessageBox::mbNo) == CMessageBox::mbrYes) {
-				std::string fname = getScreenshotName(m_movieSelectionHandler->file.Name, S_ISDIR(m_movieSelectionHandler->file.Mode));
-				if (!fname.empty())
-					unlink(fname.c_str());
-				refresh();
-			}
-#else
-			std::string fname = m_movieSelectionHandler->file.Name.c_str();
-			int ext_pos = 0;
-			ext_pos = fname.rfind('.');
-			if( ext_pos > 0) {
-				std::string extension;
-				extension = fname.substr(ext_pos + 1, fname.length() - ext_pos);
-				extension = "." + extension;
-				strReplace(fname, extension.c_str(), ".jpg");
-			}
-			printf("TMDB: %s : %s\n",m_movieSelectionHandler->file.Name.c_str(),fname.c_str());
-			if (!access(fname, F_OK)) {
-				if (ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_MOVIEBROWSER_DELETE_SCREENSHOT, CMessageBox::mbrNo, CMessageBox:: mbYes | CMessageBox::mbNo) == CMessageBox::mbrYes) {
-					if (!fname.empty())
-						unlink(fname.c_str());
-						refresh();
-				}
-			} else {
-				cTmdb* tmdb = new cTmdb(m_movieSelectionHandler->epgTitle);
-				if ((tmdb->getResults() > 0) && (tmdb->hasCover())) {
-					if (!fname.empty())
-						if (tmdb->getSmallCover(fname))
-							refresh();
-				}
-				if (tmdb)
-					delete tmdb;
-			}
-#endif
-		}
-	}
 	else
 	{
 		//TRACE("[mb]->onButtonPressMainFrame none\n");