Fixed NavigationView module selection
This commit is contained in:
@@ -6,6 +6,7 @@ import android.support.v4.app.FragmentManager;
|
|||||||
import android.support.v4.app.FragmentTransaction;
|
import android.support.v4.app.FragmentTransaction;
|
||||||
import android.os.Bundle;
|
import android.os.Bundle;
|
||||||
import android.support.annotation.NonNull;
|
import android.support.annotation.NonNull;
|
||||||
|
import android.util.Pair;
|
||||||
import android.view.View;
|
import android.view.View;
|
||||||
import android.support.design.widget.NavigationView;
|
import android.support.design.widget.NavigationView;
|
||||||
import android.support.v4.view.GravityCompat;
|
import android.support.v4.view.GravityCompat;
|
||||||
@@ -238,12 +239,11 @@ public class MainActivity extends AppCompatActivity
|
|||||||
log.d("Modules.get", success.size());
|
log.d("Modules.get", success.size());
|
||||||
//SubMenu moduleMenu = navigationView.getMenu().findItem(R.id.nav_modules).getSubMenu();
|
//SubMenu moduleMenu = navigationView.getMenu().findItem(R.id.nav_modules).getSubMenu();
|
||||||
int i = 0;
|
int i = 0;
|
||||||
for (Iterator<Modules.Module> it = success.latestSemesterIterator(); it.hasNext(); ) {
|
for (Iterator<Pair<Modules.Module, Integer>> it = success.latestSemesterIterator(); it.hasNext(); ) {
|
||||||
Modules.Module module = it.next();
|
Pair<Modules.Module, Integer> module = it.next();
|
||||||
MenuItem menuItem = mNavigationView.getMenu().add(Menu.NONE, Menu.NONE, 101 + i, module.title);
|
MenuItem menuItem = mNavigationView.getMenu().add(Menu.NONE, Menu.NONE, 101 + i, module.first.title);
|
||||||
final int finalI = i;
|
|
||||||
menuItem.setOnMenuItemClickListener(item -> {
|
menuItem.setOnMenuItemClickListener(item -> {
|
||||||
onModulesFragmentInteraction(finalI);
|
onModulesFragmentInteraction(module.second);
|
||||||
return false;
|
return false;
|
||||||
});
|
});
|
||||||
i++;
|
i++;
|
||||||
|
|||||||
@@ -2,6 +2,7 @@ package de.sebse.fuplanner.services.KVV.types;
|
|||||||
|
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.support.annotation.NonNull;
|
import android.support.annotation.NonNull;
|
||||||
|
import android.util.Pair;
|
||||||
|
|
||||||
import java.io.FileInputStream;
|
import java.io.FileInputStream;
|
||||||
import java.io.FileOutputStream;
|
import java.io.FileOutputStream;
|
||||||
@@ -27,8 +28,8 @@ import de.sebse.fuplanner.tools.logging.Logger;
|
|||||||
public class Modules implements Iterable<Modules.Module>, Serializable {
|
public class Modules implements Iterable<Modules.Module>, Serializable {
|
||||||
private final SortedModuleList list;
|
private final SortedModuleList list;
|
||||||
private String latestSemester = null;
|
private String latestSemester = null;
|
||||||
private String username = null;
|
private String username;
|
||||||
private transient Logger log = new Logger(this);
|
//private transient Logger log = new Logger(this);
|
||||||
private static final String FILE_NAME = "ModuleListSaving";
|
private static final String FILE_NAME = "ModuleListSaving";
|
||||||
|
|
||||||
public Modules(String username) {
|
public Modules(String username) {
|
||||||
@@ -107,8 +108,8 @@ public class Modules implements Iterable<Modules.Module>, Serializable {
|
|||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
public Iterator<Module> latestSemesterIterator() {
|
public Iterator<Pair<Module, Integer>> latestSemesterIterator() {
|
||||||
return new Iterator<Module>() {
|
return new Iterator<Pair<Module, Integer>>() {
|
||||||
private int index = -1;
|
private int index = -1;
|
||||||
private int next = -1;
|
private int next = -1;
|
||||||
@Override
|
@Override
|
||||||
@@ -119,13 +120,13 @@ public class Modules implements Iterable<Modules.Module>, Serializable {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public Module next() {
|
public Pair<Module, Integer> next() {
|
||||||
if (index == next)
|
if (index == next)
|
||||||
predict();
|
predict();
|
||||||
if (next == -1)
|
if (next == -1)
|
||||||
return null;
|
return null;
|
||||||
index = next;
|
index = next;
|
||||||
return list.get(index);
|
return new Pair<>(list.get(index), index);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|||||||
Reference in New Issue
Block a user