Skip to content

Commit

Permalink
Renamed weekplan_selector_bloc.dart to weekplans_bloc.dart, so that i…
Browse files Browse the repository at this point in the history
…t is consistent with its usages and the name of the file is consistent with the name of its class WeekPlans
  • Loading branch information
Frederik Bendix committed Dec 13, 2023
1 parent 4d79505 commit 208aa9d
Show file tree
Hide file tree
Showing 13 changed files with 19 additions and 15 deletions.
2 changes: 1 addition & 1 deletion lib/blocs/edit_weekplan_bloc.dart
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import 'package:api_client/models/displayname_model.dart';
import 'package:api_client/models/week_model.dart';
import 'package:flutter/material.dart';
import 'package:weekplanner/blocs/new_weekplan_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';

///This bloc is used to control the information when editing week plan
class EditWeekplanBloc extends NewWeekplanBloc {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -328,21 +328,23 @@ class WeekplansBloc extends BlocBase {
}
}



/// This helper-method deletes the given week model from the database
void _deleteWeekFromDatabase(List<WeekModel> weekModels, WeekModel weekModel) {
_api.week
.delete(_user.id!, weekModel.weekYear, weekModel.weekNumber)
.listen((bool deleted) {
if (deleted) {
deleteWeekFromStream(weekModels, weekModel);
_deleteWeekFromStream(weekModels, weekModel);
}
});
}

///This method was created during refactoring to simplify unit-testing.
/// In this way it is possible to test removing it from the stream independently
/// of testing working with the database
void deleteWeekFromStream(List<WeekModel> weekModels, WeekModel weekModel) {
void _deleteWeekFromStream(List<WeekModel> weekModels, WeekModel weekModel) {
weekModels.remove(weekModel);
_weekModels.add(weekModels);
}
Expand Down Expand Up @@ -394,7 +396,9 @@ class WeekplansBloc extends BlocBase {
_editMode.add(!_editMode.value);
}

/// This stream checks that you have only marked one week model

/// This stream checks that you have only marked one week model. It is currently
/// not being used
Stream<bool> onlyOneModelMarkedStream() {
return _markedWeekModels.map((List<WeekModel> event) => event.length == 1);
}
Expand Down
2 changes: 1 addition & 1 deletion lib/bootstrap.dart
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import 'package:weekplanner/blocs/timer_bloc.dart';
import 'package:weekplanner/blocs/toolbar_bloc.dart';
import 'package:weekplanner/blocs/upload_from_gallery_bloc.dart';
import 'package:weekplanner/blocs/weekplan_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/providers/environment_provider.dart' as environment;

Expand Down
2 changes: 1 addition & 1 deletion lib/screens/edit_weekplan_screen.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import 'package:api_client/models/displayname_model.dart';
import 'package:api_client/models/week_model.dart';
import 'package:flutter/material.dart';
import 'package:weekplanner/blocs/edit_weekplan_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/exceptions/custom_exceptions.dart';
import 'package:weekplanner/routes.dart';
Expand Down
2 changes: 1 addition & 1 deletion lib/screens/weekplan_selector_screen.dart
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import 'package:api_client/models/week_model.dart';
import 'package:auto_size_text/auto_size_text.dart';
import 'package:flutter/material.dart';
import 'package:weekplanner/blocs/pictogram_image_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/models/enums/app_bar_icons_enum.dart';
import 'package:weekplanner/routes.dart';
Expand Down
2 changes: 1 addition & 1 deletion test/blocs/edit_weekplan_bloc_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import 'package:flutter_test/flutter_test.dart';
import 'package:mocktail/mocktail.dart';
import 'package:rxdart/rxdart.dart' as rx_dart;
import 'package:weekplanner/blocs/edit_weekplan_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';

class MockWeekApi extends Mock implements WeekApi {}
Expand Down
2 changes: 1 addition & 1 deletion test/blocs/new_weekplan_bloc_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import 'package:async_test/async_test.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:mocktail/mocktail.dart';
import 'package:weekplanner/blocs/new_weekplan_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';

class MockWeekApi extends Mock implements WeekApi {}
Expand Down
2 changes: 1 addition & 1 deletion test/blocs/weekplans_bloc_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import 'package:csv/csv.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:mocktail/mocktail.dart';
import 'package:rxdart/rxdart.dart' as rx_dart;
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';

class MockWeekApi extends Mock implements WeekApi {}

Expand Down
2 changes: 1 addition & 1 deletion test/screens/copy_resolve_screen_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import 'package:weekplanner/blocs/edit_weekplan_bloc.dart';
import 'package:weekplanner/blocs/pictogram_image_bloc.dart';
import 'package:weekplanner/blocs/settings_bloc.dart';
import 'package:weekplanner/blocs/toolbar_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/screens/copy_resolve_screen.dart';
import 'package:weekplanner/screens/weekplan_selector_screen.dart';
Expand Down
2 changes: 1 addition & 1 deletion test/screens/copy_to_citizens_screen_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import 'package:weekplanner/blocs/edit_weekplan_bloc.dart';
import 'package:weekplanner/blocs/pictogram_image_bloc.dart';
import 'package:weekplanner/blocs/settings_bloc.dart';
import 'package:weekplanner/blocs/toolbar_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/screens/copy_to_citizens_screen.dart';
import 'package:weekplanner/widgets/giraf_3button_dialog.dart';
Expand Down
2 changes: 1 addition & 1 deletion test/screens/edit_weekplan_screen_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import 'package:weekplanner/blocs/edit_weekplan_bloc.dart';
import 'package:weekplanner/blocs/pictogram_bloc.dart';
import 'package:weekplanner/blocs/pictogram_image_bloc.dart';
import 'package:weekplanner/blocs/toolbar_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/screens/edit_weekplan_screen.dart';
import 'package:weekplanner/screens/pictogram_search_screen.dart';
Expand Down
2 changes: 1 addition & 1 deletion test/screens/new_weekplan_screen_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import 'package:weekplanner/blocs/new_weekplan_bloc.dart';
import 'package:weekplanner/blocs/pictogram_bloc.dart';
import 'package:weekplanner/blocs/pictogram_image_bloc.dart';
import 'package:weekplanner/blocs/toolbar_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/screens/new_weekplan_screen.dart';
import 'package:weekplanner/screens/pictogram_search_screen.dart';
Expand Down
2 changes: 1 addition & 1 deletion test/screens/weekplan_selector_screen_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import 'package:weekplanner/blocs/settings_bloc.dart';
import 'package:weekplanner/blocs/timer_bloc.dart';
import 'package:weekplanner/blocs/toolbar_bloc.dart';
import 'package:weekplanner/blocs/weekplan_bloc.dart';
import 'package:weekplanner/blocs/weekplan_selector_bloc.dart';
import 'package:weekplanner/blocs/weekplans_bloc.dart';
import 'package:weekplanner/di.dart';
import 'package:weekplanner/models/enums/app_bar_icons_enum.dart';
import 'package:weekplanner/screens/copy_resolve_screen.dart';
Expand Down

0 comments on commit 208aa9d

Please sign in to comment.