diff --git a/app/src/main/java/gr/thmmy/mthmmy/activities/BaseActivity.java b/app/src/main/java/gr/thmmy/mthmmy/activities/BaseActivity.java index c7e2fadd..b6251643 100644 --- a/app/src/main/java/gr/thmmy/mthmmy/activities/BaseActivity.java +++ b/app/src/main/java/gr/thmmy/mthmmy/activities/BaseActivity.java @@ -206,8 +206,8 @@ public class BaseActivity extends AppCompatActivity else loginLogoutItem.withName(R.string.logout).withIcon(logoutIcon); //Swap login with logout - ProfileDrawerItem p = new ProfileDrawerItem().withName(sessionManager.getUsername()); //TODO: set profile picture - accountHeader.updateProfile(p); + profileDrawerItem.withName(sessionManager.getUsername()); //TODO: set profile picture + accountHeader.updateProfile(profileDrawerItem); drawer.updateItem(loginLogoutItem); } diff --git a/app/src/main/java/gr/thmmy/mthmmy/activities/LoginActivity.java b/app/src/main/java/gr/thmmy/mthmmy/activities/LoginActivity.java index b70424c5..52c69e07 100644 --- a/app/src/main/java/gr/thmmy/mthmmy/activities/LoginActivity.java +++ b/app/src/main/java/gr/thmmy/mthmmy/activities/LoginActivity.java @@ -36,6 +36,8 @@ public class LoginActivity extends BaseActivity { //Other variables private static final String TAG = "LoginActivity"; + private LoginTask loginTask; + @Override public void onCreate(Bundle savedInstanceState) { @@ -65,7 +67,8 @@ public class LoginActivity extends BaseActivity { } //Login user - new LoginTask().execute(username, password); + loginTask = new LoginTask(); + loginTask.execute(username, password); } }); @@ -89,6 +92,9 @@ public class LoginActivity extends BaseActivity { public void onBackPressed() { // Disable going back to the MainActivity moveTaskToBack(true); +// if(loginTask!=null && loginTask.getStatus() == AsyncTask.Status.RUNNING){ TODO +// loginTask.cancel(true); +// } } private void onLoginFailed() { @@ -192,6 +198,15 @@ public class LoginActivity extends BaseActivity { loginContent.setVisibility(View.VISIBLE); spinner.setVisibility(View.INVISIBLE); } + + @Override + protected void onCancelled() { + super.onCancelled(); + btnLogin.setEnabled(true); //Re-enable login button + loginContent.setVisibility(View.VISIBLE); + spinner.setVisibility(View.INVISIBLE); + } + } //---------------------------------------LOGIN ENDS------------------------------------------------- } \ No newline at end of file diff --git a/app/src/main/java/gr/thmmy/mthmmy/session/SessionManager.java b/app/src/main/java/gr/thmmy/mthmmy/session/SessionManager.java index 4e3175bc..2bd02620 100644 --- a/app/src/main/java/gr/thmmy/mthmmy/session/SessionManager.java +++ b/app/src/main/java/gr/thmmy/mthmmy/session/SessionManager.java @@ -3,7 +3,6 @@ package gr.thmmy.mthmmy.session; import android.content.SharedPreferences; import android.graphics.Bitmap; import android.os.Environment; -import android.util.Log; import com.franmontiel.persistentcookiejar.PersistentCookieJar; import com.franmontiel.persistentcookiejar.persistence.SharedPrefsCookiePersistor; @@ -17,6 +16,7 @@ import java.io.File; import java.io.FileNotFoundException; import java.io.FileOutputStream; import java.io.IOException; +import java.io.InterruptedIOException; import java.util.List; import java.util.regex.Matcher; import java.util.regex.Pattern; @@ -48,8 +48,9 @@ public class SessionManager public static final int FAILURE = 1; //Generic Error public static final int WRONG_USER = 2; public static final int WRONG_PASSWORD = 3; - public static final int CONNECTION_ERROR = 4; - public static final int EXCEPTION = 5; + public static final int CANCELLED = 4; + public static final int CONNECTION_ERROR = 5; + public static final int EXCEPTION = 6; //Login status codes public static final int LOGGED_OUT = 0; @@ -84,7 +85,7 @@ public class SessionManager */ public int login(String... strings) { - Log.i(TAG, "Logging in..."); + Report.i(TAG, "Logging in..."); //Build the login request for each case Request request; @@ -120,7 +121,7 @@ public class SessionManager Element logoutButton = document.getElementById("logoutbtn"); //Attempt to find logout button if (logoutButton != null) //If logout button exists, login was successful { - Log.i(TAG, "Login successful!"); + Report.i(TAG, "Login successful!"); setPersistentCookieSession(); //Store cookies //Edit SharedPreferences, save session's data @@ -132,18 +133,18 @@ public class SessionManager } else { - Log.i(TAG, "Login failed."); + Report.i(TAG, "Login failed."); //Investigate login failure Elements error = document.select("b:contains(That username does not exist.)"); if (error.size() == 1) { //Wrong username - Log.i(TAG, "Wrong Username"); + Report.i(TAG, "Wrong Username"); return WRONG_USER; } error = document.select("body:contains(Password incorrect)"); if (error.size() == 1) { //Wrong password - Log.i(TAG, "Wrong Password"); + Report.i(TAG, "Wrong Password"); return WRONG_PASSWORD; } @@ -153,35 +154,39 @@ public class SessionManager } //Handle exception } + catch (InterruptedIOException e){ + Report.i(TAG, "Login InterruptedIOException: "+ e.getMessage(), e); //users cancels LoginTask + return CANCELLED; + } catch (IOException e) { - Log.w(TAG, "Login IOException: "+ e.getMessage(), e); + Report.w(TAG, "Login IOException: "+ e.getMessage(), e); return CONNECTION_ERROR; } catch (Exception e) { - Log.w(TAG, "Login Exception (other): "+ e.getMessage(), e); + Report.w(TAG, "Login Exception (other): "+ e.getMessage(), e); return EXCEPTION; } } /** * A function that checks the validity of the current saved session (if it exists). - * If LOGIN_STATUS is true, it will call login() with cookies. This can only return - * the codes {SUCCESS, FAILURE, CONNECTION_ERROR, EXCEPTION}. CONNECTION_ERROR and EXCEPTION - * are simply considered a SUCCESS (e.g. no internet connection), at least until a more - * thorough handling of different exceptions is implemented (if considered mandatory). + * If LOGIN_STATUS is true, it will call login() with cookies. On failure, this can only return + * the code FAILURE. CANCELLED, CONNECTION_ERROR and EXCEPTION are simply considered a SUCCESS + * (e.g. no internet connection), at least until a more thorough handling of different + * exceptions is implemented (if considered mandatory). * Always call it in a separate thread in a way that won't hinder performance (e.g. after * fragments' data are retrieved). */ public void validateSession() { - Log.i(TAG, "Validating session..."); + Report.i(TAG, "Validating session..."); //Check if user is currently logged in int status = sharedPrefs.getInt(LOGIN_STATUS,LOGGED_OUT); if(status==LOGGED_IN) { int loginResult = login(); - if(loginResult == SUCCESS || loginResult == CONNECTION_ERROR || loginResult == EXCEPTION) + if(loginResult != FAILURE) return; } else if(status==AS_GUEST) @@ -195,7 +200,7 @@ public class SessionManager */ public void guestLogin() { - Log.i("TAG", "Continuing as a guest, as chosen by the user."); + Report.i("TAG", "Continuing as a guest, as chosen by the user."); clearSessionData(); sharedPrefs.edit().putInt(LOGIN_STATUS, AS_GUEST).apply(); } @@ -206,7 +211,7 @@ public class SessionManager */ public int logout() { - Log.i(TAG, "Logging out..."); + Report.i(TAG, "Logging out..."); Request request = new Request.Builder() .url(sharedPrefs.getString(LOGOUT_LINK,"LogoutLink")) @@ -220,17 +225,17 @@ public class SessionManager Elements loginButton = document.select("[value=Login]"); //Attempt to find login button if (!loginButton.isEmpty()) //If login button exists, logout was successful { - Log.i("Logout", "Logout successful!"); + Report.i(TAG, "Logout successful!"); return SUCCESS; } else { - Log.i(TAG, "Logout failed."); + Report.i(TAG, "Logout failed."); return FAILURE; } } catch (IOException e) { - Log.w(TAG, "Logout IOException: "+ e.getMessage(), e); + Report.w(TAG, "Logout IOException: "+ e.getMessage(), e); return CONNECTION_ERROR; } catch (Exception e) { - Log.w(TAG, "Logout Exception: "+ e.getMessage(), e); + Report.w(TAG, "Logout Exception: "+ e.getMessage(), e); return EXCEPTION; } finally { //All data should always be cleared from device regardless the result of logout @@ -279,7 +284,7 @@ public class SessionManager sharedPrefs.edit().clear().apply(); //Clear session data sharedPrefs.edit().putString(USERNAME, guestName).apply(); sharedPrefs.edit().putInt(LOGIN_STATUS, LOGGED_OUT).apply(); //User logs out - Log.i(TAG,"Session data cleared."); + Report.i(TAG,"Session data cleared."); } private String extractUserName(Document doc) @@ -310,7 +315,7 @@ public class SessionManager File pictureFile = getOutputMediaFile(package_name, image_name); if (pictureFile == null) { - Log.d(TAG, + Report.d(TAG, "Error creating media file, check storage permissions: ");// e.getMessage()); return; } @@ -321,9 +326,9 @@ public class SessionManager bitmap.compress(Bitmap.CompressFormat.PNG, 90, fos); fos.close(); } catch (FileNotFoundException e) { - Log.d(TAG, "File not found: " + e.getMessage()); + Report.d(TAG, "File not found: " + e.getMessage()); } catch (IOException e) { - Log.d(TAG, "Error accessing file: " + e.getMessage()); + Report.d(TAG, "Error accessing file: " + e.getMessage()); } returnImage[0] = pictureFile; } @@ -362,4 +367,4 @@ public class SessionManager //----------------------------------OTHER FUNCTIONS END----------------------------------------- -} +} \ No newline at end of file