Merge pull request #73 from Z3r0byte/dev

Dev

Author: Z3r0byte

app/build.gradle

@@ -23,8 +23,8 @@ android {
         applicationId "com.z3r0byte.magistify"
         minSdkVersion 16
         targetSdkVersion 26
-        versionCode 212
-        versionName "2.1.2"
+        versionCode 213
+        versionName "2.1.3"
         testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
     }
     buildTypes {

app/src/main/java/com/z3r0byte/magistify/AppointmentDetailsActivity.java

@@ -40,6 +40,7 @@
 
 import net.ilexiconn.magister.Magister;
 import net.ilexiconn.magister.container.Appointment;
+import net.ilexiconn.magister.container.type.AppointmentType;
 import net.ilexiconn.magister.container.type.InfoType;
 import net.ilexiconn.magister.handler.AppointmentHandler;
 import net.ilexiconn.magister.util.DateUtil;
@@ -142,15 +143,18 @@ private void setupDetailscard(Appointment appointment) {
         periodImageView.setImageDrawable(drawable);
         periodTextInput.setText(appointment.periodFrom + "");
 
-
-        try {
-            String teacher = appointment.teachers[0].name;
-            drawable = new IconicsDrawable(this, GoogleMaterial.Icon.gmd_person);
-            teacherLayout.setVisibility(View.VISIBLE);
-            teacherImageView.setImageDrawable(drawable);
-            teacherTextInput.setText(teacher);
-        } catch (ArrayIndexOutOfBoundsException e) {
-            e.printStackTrace();
+        if (appointment.type != AppointmentType.PERSONAL) {
+            try {
+                String teacher = appointment.teachers[0].name;
+                drawable = new IconicsDrawable(this, GoogleMaterial.Icon.gmd_person);
+                teacherLayout.setVisibility(View.VISIBLE);
+                teacherImageView.setImageDrawable(drawable);
+                teacherTextInput.setText(teacher);
+            } catch (ArrayIndexOutOfBoundsException e) {
+                e.printStackTrace();
+            }
+        } else {
+            teacherLayout.setVisibility(View.GONE);
         }
 
 
@@ -244,6 +248,10 @@ public void run() {
             }
         });
 
+        if (appointment.type == AppointmentType.PERSONAL) {
+            lastEdited.setVisibility(View.GONE);
+            return;
+        }
         lastEdited.setText(String.format(getString(R.string.msg_last_edited), getString(R.string.msg_loading)));
         new Thread(new Runnable() {
             @Override

app/src/main/java/com/z3r0byte/magistify/SetupActivity.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016-2017 Bas van den Boom 'Z3r0byte'
+ * Copyright (c) 2016-2018 Bas van den Boom 'Z3r0byte'
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -65,6 +65,13 @@ protected void onCreate(Bundle savedInstanceState) {
                 .backgroundDark(R.color.setup_color_1)
                 .build());
 
+        addSlide(new SimpleSlide.Builder()
+                .title(R.string.setup_title_4)
+                .description(R.string.setup_desc_4)
+                .background(R.color.setup_color_1)
+                .backgroundDark(R.color.setup_color_1)
+                .build());
+
         final FragmentSlide searchSlide = new FragmentSlide.Builder()
                 .background(R.color.setup_color_3)
                 .backgroundDark(R.color.setup_color_3)
@@ -98,7 +105,7 @@ public void onNavigationBlocked(int position, int direction) {
         addOnPageChangeListener(new ViewPager.OnPageChangeListener() {
             @Override
             public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {
-                if (positionOffset == 0 && position == 5) {
+                if (positionOffset == 0 && position == 6) {
                     Intent intent = new Intent(getApplicationContext(), StartActivity.class);
                     startActivity(intent);
                     finish();

app/src/main/res/values/strings.xml

@@ -143,10 +143,12 @@
     <string name="setup_desc_3">Magistify is GEEN volledige Magister app. Het biedt slechts handige tooltjes aan die andere apps
      niet hebben. Plaats dus aub geen reviews in de richting van "Ik kan niet al mijn cijfers zien!". Wie weet wordt Magistify
     ooit nog een volledige app, maar nu is hij dat nog niet.</string>
+    <string name="setup_desc_4">Op sommige Android-versies van sommige fabrikanten (o.a. OnePlus) werken de meldingen niet goed. Dit komt omdat zij achtergrondapps afsluiten
+        als er te veel zijn. Ik kan hier helaas weinig aan doen, dus geef alsjeblieft geen slechte review als de meldingen niet werken.</string>
     <string name="setup_title_1">Welkom bij Magistify</string>
     <string name="setup_title_2">Permissies</string>
     <string name="setup_title_3">Even om onduidelijkheid te voorkomen:</string>
-    <string name="setup_title_4"></string>
+    <string name="setup_title_4">Belangrijk:</string>
     <string name="setup_title_5"></string>
 
     <string name="snackbar_fix">Fix</string>
@@ -189,8 +191,8 @@
     <string name="err_failed_login_dashboard">Kon niet inloggen, gebruikersnaam/wachtwoord mogelijk verkeerd.</string>
     <string name="title_activity_homework">Huiswerk</string>
     <string name="symbol_click">{gmd-chevron-right}</string>
-    <string name="title_magistify_changelog">Magistify 2.1.1</string>
-    <string name="desc_magistify_changelog">Bug-fixes:\n- Crash als een nieuw cijfer een \' bevatte in de omschrijving. (#70)\n- De in-app aankopen service wordt ontbonden als de app wordt afgesloten</string>
+    <string name="title_magistify_changelog">Magistify 2.1.3</string>
+    <string name="desc_magistify_changelog">Bug-fixes:\n- Crash wanneer je een eigen afspraak opende.</string>
     <string name="msg_test_name">Toets:</string>
     <string name="msg_subject">Vak:</string>
     <string name="msg_date_submitted">Ingevoerd op:</string>