Skip to content

Commit fa68e57

Browse files
committed
refactor(content_management): simplify BlocConsumer listenWhen condition
- Remove unnecessary conditions in listenWhen, as status change is sufficient - Adjust state update logic to only trigger on initial status
1 parent 8379096 commit fa68e57

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

lib/content_management/view/edit_headline_page.dart

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -100,12 +100,7 @@ class _EditHeadlineViewState extends State<_EditHeadlineView> {
100100
],
101101
),
102102
body: BlocConsumer<EditHeadlineBloc, EditHeadlineState>(
103-
listenWhen: (previous, current) =>
104-
previous.status != current.status ||
105-
previous.title != current.title ||
106-
previous.excerpt != current.excerpt ||
107-
previous.url != current.url ||
108-
previous.imageUrl != current.imageUrl,
103+
listenWhen: (previous, current) => previous.status != current.status,
109104
listener: (context, state) {
110105
if (state.status == EditHeadlineStatus.success &&
111106
state.updatedHeadline != null &&
@@ -128,8 +123,7 @@ class _EditHeadlineViewState extends State<_EditHeadlineView> {
128123
);
129124
}
130125
// Update text controllers when data is loaded or changed
131-
if (state.status == EditHeadlineStatus.initial ||
132-
state.status == EditHeadlineStatus.success) {
126+
if (state.status == EditHeadlineStatus.initial) {
133127
_titleController.text = state.title;
134128
_excerptController.text = state.excerpt;
135129
_urlController.text = state.url;

0 commit comments

Comments
 (0)