diff --git a/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadActivity.java b/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadActivity.java index 08f3b891..207b2099 100644 --- a/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadActivity.java +++ b/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadActivity.java @@ -148,19 +148,22 @@ public class UploadActivity extends BaseActivity { } String maybeSemester = (String) ((AppCompatSpinnerWithoutDefault) - categoriesSpinners.getChildAt(numberOfSpinners - 2)).getSelectedItem(), - maybeCourse = (String) ((AppCompatSpinnerWithoutDefault) - categoriesSpinners.getChildAt(numberOfSpinners - 1)).getSelectedItem(); + categoriesSpinners.getChildAt(numberOfSpinners - 2)).getSelectedItem(); + String maybeCourse = (String) ((AppCompatSpinnerWithoutDefault) + categoriesSpinners.getChildAt(numberOfSpinners - 1)).getSelectedItem(); if (!maybeSemester.contains("εξάμηνο") && !maybeSemester.contains("Εξάμηνο")) { Toast.makeText(view.getContext(), "Please choose a course category", Toast.LENGTH_SHORT).show(); return; } + if (maybeCourse == null) { + Toast.makeText(view.getContext(), "Please choose a course", Toast.LENGTH_SHORT).show(); + return; + } //Fixes course and semester maybeCourse = maybeCourse.replaceAll("-", "").replace("(ΝΠΣ)", "").trim(); - maybeSemester = maybeSemester.replaceAll("-", "").trim(). - substring(0, 1); + maybeSemester = maybeSemester.replaceAll("-", "").trim().substring(0, 1); Intent intent = new Intent(UploadActivity.this, UploadFieldsBuilderActivity.class); Bundle extras = new Bundle(); @@ -359,7 +362,11 @@ public class UploadActivity extends BaseActivity { @Override public void onActivityResult(int requestCode, int resultCode, Intent data) { - if (requestCode == REQUEST_CODE_CHOOSE_FILE && resultCode == Activity.RESULT_OK && data != null) { + if (requestCode == REQUEST_CODE_CHOOSE_FILE && data != null) { + if (resultCode == Activity.RESULT_OK) { + return; + } + fileUri = data.getData(); if (fileUri != null) { String filename = filenameFromUri(fileUri); @@ -436,7 +443,7 @@ public class UploadActivity extends BaseActivity { if (!tempDirectory.exists()) { if (!tempDirectory.mkdirs()) { - //TODO timber message? + Timber.w("Temporary directory build returned false in %s", UploadActivity.class.getSimpleName()); Toast.makeText(this, "Couldn't create temporary directory", Toast.LENGTH_SHORT).show(); return null; } @@ -449,7 +456,7 @@ public class UploadActivity extends BaseActivity { try { inputStream = getContentResolver().openInputStream(fileUri); if (inputStream == null) { - //TODO timber message? + Timber.w("Input stream was null, %s", UploadActivity.class.getSimpleName()); return null; } diff --git a/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadFieldsBuilderActivity.java b/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadFieldsBuilderActivity.java index 055971df..73bf6962 100644 --- a/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadFieldsBuilderActivity.java +++ b/app/src/main/java/gr/thmmy/mthmmy/activities/upload/UploadFieldsBuilderActivity.java @@ -217,6 +217,7 @@ public class UploadFieldsBuilderActivity extends AppCompatActivity { @Nullable private String getGreeklishOrMinifiedCourseName(boolean greeklish) { + //TODO fill missing values if (course.contains("Συστήματα Υπολογιστών (Υπολογιστικά Συστήματα)")) { return greeklish ? "sys_ypologistwn" : "Συσ. Υπολογιστών"; } else if (course.contains("Τεχνική Μηχανική")) { @@ -305,6 +306,8 @@ public class UploadFieldsBuilderActivity extends AppCompatActivity { return greeklish ? "SHE_I" : "ΣΗΕ 1"; } else if (course.contains("Υψηλές Τάσεις Ι")) { return greeklish ? "ypshles_I" : "Υψηλές 1"; + } else if (course.contains("Θεωρία και Τεχνολογία Πυρηνικών Αντιδραστήρων")) { + return greeklish ? "texn_antidrasthrwn" : "Τεχνολογία Αντιδραστήρων"; } else if (course.contains("Τεχνολογία Ηλεκτροτεχνικών Υλικών")) { return greeklish ? "Hlektrotexnika_Ylika" : "Ηλεκτροτεχνικά Υλικά"; } else if (course.contains("Ηλεκτρικές Μηχανές Α'")) { @@ -315,8 +318,12 @@ public class UploadFieldsBuilderActivity extends AppCompatActivity { return greeklish ? "SHE_II" : "ΣΗΕ 2"; } else if (course.contains("Υψηλές Τάσεις ΙΙ")) { return greeklish ? "ypshles_II" : "Υψηλές 2"; + } else if (course.contains("Αρχές Οικονομίας")) { + return greeklish ? "arx_oikonomias" : "Αρχές Οικονομίας"; + } else if (course.contains("Διανεμημένη Παραγωγή")) { + return greeklish ? "dian_paragwgh" : "Διανεμημένη Παραγωγή"; } else if (course.contains("Διαχείριση Συστημάτων Ηλεκτρικής Ενέργειας")) { - return greeklish ? "ymes" : "ΥΜΕΣ"; + return greeklish ? "dshe" : "ΔΣΗΕ"; } else if (course.contains("Υψηλές Τάσεις ΙΙΙ")) { return greeklish ? "ypshles_III" : "Υψηλές 3"; } else if (course.contains("Ανάλυση Συστημάτων Ηλεκτρικής Ενέργειας")) { @@ -327,10 +334,18 @@ public class UploadFieldsBuilderActivity extends AppCompatActivity { return greeklish ? "isxyos_I" : "Ισχύος 1"; } else if (course.contains("Συστήματα Ηλεκτρικής Ενέργειας ΙΙΙ")) { return greeklish ? "SHE_III" : "ΣΗΕ 3"; + } else if (course.contains("Σερβοκινητήρια Συστήματα")) { + return greeklish ? "servo" : "Σέρβο"; + } else if (course.contains("Συστήματα Ηλεκτροκίνησης")) { + return greeklish ? "hlektrokinhsh" : "Ηλεκτροκίνηση"; } else if (course.contains("Υπολογιστικές Μέθοδοι στα Ενεργειακά Συστήματα")) { return greeklish ? "ymes" : "ΥΜΕΣ"; } else if (course.contains("Υψηλές Τάσεις 4")) { return greeklish ? "ypshles_IV" : "Υψηλές 4"; + } else if (course.contains("Ηλεκτρικές Μηχανές Γ'")) { + return greeklish ? "mhxanes_C" : "Μηχανές Γ"; + } else if (course.contains("Ηλεκτρική Οικονομία")) { + return greeklish ? "hlektr_oikonomia" : "Ηλεκτρική Οικονομία"; } else if (course.contains("Ηλεκτρονικά Ισχύος ΙΙ")) { return greeklish ? "isxyos_II" : "Ισχύος 2"; } else if (course.contains("Ανάλυση και Σχεδίαση Αλγορίθμων")) { @@ -435,6 +450,8 @@ public class UploadFieldsBuilderActivity extends AppCompatActivity { return greeklish ? "PSES" : "ΨΕΣ"; } else if (course.contains("Εισαγωγή στην Πολιτική Οικονομία")) { return greeklish ? "polit_oik" : "Πολιτική Οικονομία"; + } else if (course.contains("Θεωρία Σκέδασης")) { + return greeklish ? "skedash" : "Σκέδαση"; } else if (course.contains("Προηγμένες Τεχνικές Επεξεργασίας Σήματος")) { return greeklish ? "ptes" : "ΠΤΕΣ"; } else if (course.contains("Τηλεοπτικά Συστήματα")) { @@ -447,6 +464,16 @@ public class UploadFieldsBuilderActivity extends AppCompatActivity { return greeklish ? "theoria_plir" : "Θεωρία Πληρ."; } else if (course.contains("Οπτικές Επικοινωνίες")) { return greeklish ? "optikes_thlep" : "Οπτικές Τηλεπ."; + } else if (course.contains("Ευρυζωνικά Δίκτυα")) { + return greeklish ? "eyryzwnika" : "Ευρυζωνικά"; + } else if (course.contains("Τεχνικές μη Καταστρεπτικών Δοκιμών")) { + return greeklish ? "non_destructive_tests" : "Μη Καταστρεπτικές Δοκιμές"; + } else if (course.contains("Φωτονική Τεχνολογία")) { + return greeklish ? "fwtonikh" : "Φωτονική"; + } else if (course.contains("Μικροκυματική Τηλεπισκόπηση")) { + return greeklish ? "thlepiskophsh" : "Τηλεπισκόπηση"; + } else if (course.contains("Μικροκύματα II")) { + return greeklish ? "mikrokymata_II" : "Μικροκύματα 2"; } else { return null; }