MarianumMobile/Client
MarianumMobile/Client
5
1
Fork 0
Code Issues 34 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

added connected double lessons with own setting #49

Merged
MineTec merged 4 commits from develop-connectedDoubleLessons into develop 2024-03-31 10:50:59 +00:00
Conversation 24 Commits 4 Files Changed 6 +48 -12
3 changed files with 48 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a0c025b58b - Show all commits

11
lib/extensions/dateTime.dart
View File

@ -16,4 +16,15 @@ extension IsSameDay on DateTime {
TimeOfDay toTimeOfDay() { TimeOfDay toTimeOfDay() {
return TimeOfDay(hour: hour, minute: minute); return TimeOfDay(hour: hour, minute: minute);
} }
bool isSameDateTime(DateTime other) {
bool isSameDay = this.isSameDay(other);
bool isSameTimeOfDay = (toTimeOfDay() == other.toTimeOfDay());
return isSameDay && isSameTimeOfDay;
}
bool isSameOrAfter(DateTime other) {
return isSameDateTime(other) || isAfter(other);
}
} }

45
lib/view/pages/timetable/timetable.dart
View File

@ -246,9 +246,9 @@ class _TimetableState extends State<Timetable> {
List<GetTimetableResponseObject> _removeDuplicates(TimetableProps data, Duration maxTimeBetweenDouble) { List<GetTimetableResponseObject> _removeDuplicates(TimetableProps data, Duration maxTimeBetweenDouble) {
List<GetTimetableResponseObject> timetableList = data.getTimetableResponse.result.toList(); List<GetTimetableResponseObject> timetableList = data.getTimetableResponse.result.toList( );
Pupsi marked this conversation as resolved
MineTec commented 2024-03-30 18:03:56 +00:00
Review
Copy Link

klammern weg oder aufklappen mit leerzeichen zwischen bedingungsklammern und block

klammern weg **oder** aufklappen mit leerzeichen zwischen bedingungsklammern und block
if(timetableList.isEmpty){ return timetableList; } if ( timetableList.isEmpty ) return timetableList;
timetableList.sort((a, b) => _parseWebuntisTimestamp(a.date, a.startTime).compareTo(_parseWebuntisTimestamp(b.date, b.startTime))); timetableList.sort((a, b) => _parseWebuntisTimestamp(a.date, a.startTime).compareTo(_parseWebuntisTimestamp(b.date, b.startTime)));
@ -256,15 +256,38 @@ class _TimetableState extends State<Timetable> {
for (var i = 1; i < timetableList.length; i++) { for (var i = 1; i < timetableList.length; i++) {
GetTimetableResponseObject currentElement = timetableList.elementAt(i); GetTimetableResponseObject currentElement = timetableList.elementAt(i);
Pupsi marked this conversation as resolved
MineTec commented 2024-03-30 18:28:02 +00:00
Review
Copy Link

das sollte umgeschrieben werden.
ggf musst du die pfeilsyntax in einen normalen curly braces block abändern und dort dann mit normalen ifs erst auf null prüfen - auch wenn das länger ist

das nullable mit dem fallback auf 1 == 2 ist sehr sehr hacky...

bool isSameSubject() {
  SomeNullableType? currentElementSubjects = currentElement.su.firstOrNull;
  SomeNullableType? previousElementSubjects = previousElement.su.firstOrNull;

  if(currentElementSubjects == null || previousElementSubjects == null) return false;
  return currentElementSubjects!.id == previousElementSubjects!.id;
}
das sollte umgeschrieben werden. ggf musst du die pfeilsyntax in einen normalen curly braces block abändern und dort dann mit normalen ifs erst auf null prüfen - auch wenn das länger ist das nullable mit dem fallback auf 1 == 2 ist sehr sehr hacky... ``` bool isSameSubject() { SomeNullableType? currentElementSubjects = currentElement.su.firstOrNull; SomeNullableType? previousElementSubjects = previousElement.su.firstOrNull; if(currentElementSubjects == null || previousElementSubjects == null) return false; return currentElementSubjects!.id == previousElementSubjects!.id; } ```
bool isSameSubject() => (currentElement.su.firstOrNull?.id ?? 1) == (previousElement.su.firstOrNull?.id ?? 2); bool isSameLesson() {
bool isNotSeparated() => _parseWebuntisTimestamp(previousElement.date, previousElement.endTime).add(maxTimeBetweenDouble).add(const Duration(seconds: 1)) int? currentSubjectId = currentElement.su.firstOrNull?.id;
.isAfter(_parseWebuntisTimestamp(currentElement.date, currentElement.startTime)); int? previousSubjectId = previousElement.su.firstOrNull?.id;
Pupsi marked this conversation as resolved
MineTec commented 2024-03-30 18:35:19 +00:00
Review
Copy Link

leerzeichen

leerzeichen
if(isSameSubject() && isNotSeparated()){ if( currentSubjectId == null || previousSubjectId == null || currentSubjectId != previousSubjectId ) return false;
int? currentRoomId = currentElement.ro.firstOrNull?.id;
int? previousRoomId = previousElement.ro.firstOrNull?.id;
if( currentRoomId != previousRoomId ) return false;
int? currentTeacherId = currentElement.te.firstOrNull?.id;
int? previousTeacherId = previousElement.te.firstOrNull?.id;
if( currentTeacherId != previousTeacherId ) return false;
String? currentStatusCode = currentElement.code;
String? previousStatusCode = previousElement.code;
if( currentStatusCode != previousStatusCode ) return false;
return true;
}
bool isNotSeparated() => _parseWebuntisTimestamp( previousElement.date, previousElement.endTime ).add( maxTimeBetweenDouble )
.isSameOrAfter( _parseWebuntisTimestamp( currentElement.date, currentElement.startTime ) );
if ( isSameLesson() && isNotSeparated() ) {
previousElement.endTime = currentElement.endTime; previousElement.endTime = currentElement.endTime;
timetableList.remove(currentElement); timetableList.remove( currentElement );
i--; i--;
}else{ } else {
previousElement = currentElement; previousElement = currentElement;
} }
} }
@ -274,10 +297,10 @@ class _TimetableState extends State<Timetable> {
TimetableEvents _buildTableEvents(TimetableProps data) { TimetableEvents _buildTableEvents(TimetableProps data) {
List<GetTimetableResponseObject> timetableList = data.getTimetableResponse.result.toList(); List<GetTimetableResponseObject> timetableList = data.getTimetableResponse.result.toList( );
if(settings.val().timetableSettings.connectDoubleLessons){ if( settings.val().timetableSettings.connectDoubleLessons ){
timetableList = _removeDuplicates(data, const Duration(minutes: 5)); timetableList = _removeDuplicates( data, const Duration(minutes: 5) );
} }
List<Appointment> appointments = timetableList.map((element) { List<Appointment> appointments = timetableList.map((element) {

4
lib/view/settings/settings.dart
View File

@ -8,6 +8,7 @@ import 'package:provider/provider.dart';
import 'package:shared_preferences/shared_preferences.dart'; import 'package:shared_preferences/shared_preferences.dart';
import '../../model/accountData.dart'; import '../../model/accountData.dart';
import '../../model/timetable/timetableProps.dart';
import '../../notification/notifyUpdater.dart'; import '../../notification/notifyUpdater.dart';
import '../../storage/base/settingsProvider.dart'; import '../../storage/base/settingsProvider.dart';
import '../../theming/appTheme.dart'; import '../../theming/appTheme.dart';
@ -100,11 +101,12 @@ class _SettingsState extends State<Settings> {
ListTile( ListTile(
leading: const Icon(Icons.calendar_view_day_outlined), leading: const Icon(Icons.calendar_view_day_outlined),
Pupsi marked this conversation as resolved
MineTec commented 2024-03-30 18:41:22 +00:00
Review
Copy Link

der Begriff ist glaube ich nicht ganz so super

was hälst du von Doppelstunden zusammenhängend anzeigen?

der Begriff ist glaube ich nicht ganz so super was hälst du von `Doppelstunden zusammenhängend anzeigen`?
title: const Text("Doppelstunden als eine Stunde anzeigen"), title: const Text('Doppelstunden zusammenhängend anzeigen'),
trailing: Checkbox( trailing: Checkbox(
value: settings.val().timetableSettings.connectDoubleLessons, value: settings.val().timetableSettings.connectDoubleLessons,
onChanged: (e) { onChanged: (e) {
Pupsi marked this conversation as resolved
MineTec commented 2024-03-30 18:59:26 +00:00
Review
Copy Link

beim testen ist mir aufgefallen, dass beim Ändern der Einstellung diese nicht direkt sichtbar sind auf dem Stundenplan.

Dies führt zu Verwirrung, wenn man z.B. erst im Stundenplan scrollen/ neuladen muss um die verbunden Stunden sehen zu können.

der Grund dahinter ist, das die meisten Einstellungen durch ein einfaches widget rebuild sichtbar werden. Die logik für die vebrunden Stundenpläne wird aber nicht mit jedem widget-build aufgerufen, da dies extrem auf die performance gehen würde.
Daher musst du das "neubauen" des Stundenplans manuell triggern.
Du kannst an dieser Stelle nach dem setzen der Einstellung das so wie beim beim drag-to-reload des stundenplans machen.

Siehe:

lib/view/pages/timetable/timetable.dart Line 177 in 75846750f7
Provider.of<TimetableProps>(context, listen: false).run(renew: true);

die Methode run() auf TimetableProps triggert den rebuilt

den parameter renew brauchst du nicht. (renew besagt das die daten nicht aus dem cache kommen dürfen und neu von webuntis heruntergeladen werden sollen)

beim testen ist mir aufgefallen, dass beim Ändern der Einstellung diese nicht direkt sichtbar sind auf dem Stundenplan. Dies führt zu Verwirrung, wenn man z.B. erst im Stundenplan scrollen/ neuladen muss um die verbunden Stunden sehen zu können. der Grund dahinter ist, das die meisten Einstellungen durch ein einfaches widget rebuild sichtbar werden. Die logik für die vebrunden Stundenpläne wird aber nicht mit jedem widget-build aufgerufen, da dies extrem auf die performance gehen würde. Daher musst du das "neubauen" des Stundenplans manuell triggern. Du kannst an dieser Stelle nach dem setzen der Einstellung das so wie beim beim drag-to-reload des stundenplans machen. Siehe: https://mhsl.eu/gitea/MarianumMobile/Client/src/commit/75846750f7f1ea4e161825e9f3f3529e47cbdb57/lib/view/pages/timetable/timetable.dart#L177 die Methode run() auf TimetableProps triggert den rebuilt den parameter renew brauchst du nicht. (renew besagt das die daten nicht aus dem cache kommen dürfen und neu von webuntis heruntergeladen werden sollen)
settings.val(write: true).timetableSettings.connectDoubleLessons = e!; settings.val(write: true).timetableSettings.connectDoubleLessons = e!;
Provider.of<TimetableProps>(context, listen: false).run(renew: false);
}, },
), ),
), ),