Feature: Edit Account Name

lib/pages/accounts.dart

@@ -199,7 +199,8 @@ class _AccountDetailsState extends State<AccountDetails>
       child: PagedListView<int, AccountRead>(
         pagingController: _pagingController,
         builderDelegate: PagedChildBuilderDelegate<AccountRead>(
-          itemBuilder: accountRowBuilder,
+          itemBuilder: (BuildContext context, AccountRead item, int index) =>
+              accountRowBuilder(context, item, index, _pagingController),
         ),
       ),
     );

lib/pages/home/accounts/row.dart

@@ -1,14 +1,28 @@
+import 'dart:convert';
+
 import 'package:animations/animations.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_gen/gen_l10n/app_localizations.dart';
 import 'package:intl/intl.dart';
+import 'package:logging/logging.dart';
+import 'package:provider/provider.dart';
+
+import 'package:chopper/chopper.dart';
+import 'package:infinite_scroll_pagination/infinite_scroll_pagination.dart';
 
+import 'package:waterflyiii/auth.dart';
 import 'package:waterflyiii/extensions.dart';
 import 'package:waterflyiii/generated/swagger_fireflyiii_api/firefly_iii.swagger.dart';
 import 'package:waterflyiii/pages/home/transactions.dart';
 import 'package:waterflyiii/widgets/fabs.dart';
 
-Widget accountRowBuilder(BuildContext context, AccountRead account, int index) {
+Widget accountRowBuilder(
+  BuildContext context,
+  AccountRead account,
+  int index,
+  PagingController<int, AccountRead> pagingController,
+) {
+  String name = account.attributes.name;
   late double currentAmount;
   if (account.attributes.type == ShortAccountTypeProperty.liability) {
     currentAmount = double.tryParse(account.attributes.currentDebt ?? "") ?? 0;
@@ -93,18 +107,10 @@ Widget accountRowBuilder(BuildContext context, AccountRead account, int index) {
       subtitle = S.of(context).generalUnknown;
   }
   return OpenContainer(
-    openBuilder: (BuildContext context, Function closedContainer) => Scaffold(
-      appBar: AppBar(
-        title: Text(account.attributes.name),
-      ),
-      floatingActionButton:
-          account.attributes.type == ShortAccountTypeProperty.asset
-              ? NewTransactionFab(
-                  context: context,
-                  accountId: account.id,
-                )
-              : null,
-      body: HomeTransactions(accountId: account.id),
+    openBuilder: (BuildContext context, Function closedContainer) =>
+        AccountTXpage(
+      account: account,
+      nameUpdateFunc: (_) => pagingController.refresh(),
     ),
     openColor: Theme.of(context).cardColor,
     closedColor: Theme.of(context).cardColor,
@@ -117,7 +123,7 @@ Widget accountRowBuilder(BuildContext context, AccountRead account, int index) {
     closedElevation: 0,
     closedBuilder: (BuildContext context, Function openContainer) => ListTile(
       title: Text(
-        account.attributes.name,
+        name,
         maxLines: 1,
         overflow: TextOverflow.ellipsis,
       ),
@@ -166,3 +172,143 @@ Widget accountRowBuilder(BuildContext context, AccountRead account, int index) {
     ),
   );
 }
+
+class AccountTXpage extends StatefulWidget {
+  const AccountTXpage({
+    super.key,
+    required this.account,
+    required this.nameUpdateFunc,
+  });
+
+  final AccountRead account;
+  final Function(String) nameUpdateFunc;
+
+  @override
+  State<AccountTXpage> createState() => _AccountTXpageState();
+}
+
+class _AccountTXpageState extends State<AccountTXpage> {
+  final Logger log = Logger("Pages.Accounts.Row.Page");
+
+  late String _name;
+  late Widget _titleWidget;
+  late Widget _editIcon;
+
+  final TextEditingController _textController = TextEditingController();
+  final FocusNode _focusNode = FocusNode();
+
+  @override
+  void initState() {
+    super.initState();
+
+    _name = widget.account.attributes.name;
+    _titleWidget = Text(_name);
+    _editIcon = IconButton(
+      icon: Icon(Icons.edit),
+      onPressed: showTextfield,
+    );
+  }
+
+  @override
+  void dispose() {
+    _textController.dispose();
+    _focusNode.dispose();
+
+    super.dispose();
+  }
+
+  void showTextfield() {
+    log.finest(() => "showing edit field");
+    setState(() {
+      _textController.text = widget.account.attributes.name;
+      _titleWidget = TextField(
+        controller: _textController,
+        focusNode: _focusNode,
+        onEditingComplete: submitTextfield,
+      );
+      _editIcon = IconButton(
+        icon: Icon(Icons.check),
+        onPressed: submitTextfield,
+      );
+
+      _focusNode.requestFocus();
+    });
+  }
+
+  void submitTextfield() async {
+    log.finest(() => "submitting edit field");
+    final ScaffoldMessengerState msg = ScaffoldMessenger.of(context);
+
+    if (_textController.text.isNotEmpty &&
+        _textController.text != widget.account.attributes.name) {
+      try {
+        final FireflyIii api = context.read<FireflyService>().api;
+        final Response<AccountSingle> response = await api.v1AccountsIdPut(
+          id: widget.account.id,
+          body: AccountUpdate(name: _textController.text),
+        );
+        if (!response.isSuccessful || response.body == null) {
+          log.severe("Error while submitting new name to API");
+          String error;
+          try {
+            ValidationErrorResponse valError = ValidationErrorResponse.fromJson(
+              json.decode(response.error.toString()),
+            );
+            error = valError.message ??
+                // ignore: use_build_context_synchronously
+                (context.mounted
+                    // ignore: use_build_context_synchronously
+                    ? S.of(context).errorUnknown
+                    : "[nocontext] Unknown error.");
+          } catch (_) {
+            // ignore: use_build_context_synchronously
+            error = context.mounted
+                // ignore: use_build_context_synchronously
+                ? S.of(context).errorUnknown
+                : "[nocontext] Unknown error.";
+          }
+
+          msg.showSnackBar(SnackBar(
+            content: Text(error),
+            behavior: SnackBarBehavior.floating,
+          ));
+          return;
+        }
+
+        _name = response.body!.data.attributes.name;
+        widget.nameUpdateFunc(_name);
+      } catch (e, stackTrace) {
+        log.severe("Error while submitting new name to API", e, stackTrace);
+      }
+    }
+
+    log.finest(() => "switching back to text field");
+    setState(() {
+      _titleWidget = Text(_name);
+      _editIcon = IconButton(
+        icon: Icon(Icons.edit),
+        onPressed: showTextfield,
+      );
+    });
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    return Scaffold(
+      appBar: AppBar(
+        title: _titleWidget,
+        actions: <Widget>[
+          _editIcon,
+        ],
+      ),
+      floatingActionButton:
+          widget.account.attributes.type == ShortAccountTypeProperty.asset
+              ? NewTransactionFab(
+                  context: context,
+                  accountId: widget.account.id,
+                )
+              : null,
+      body: HomeTransactions(accountId: widget.account.id),
+    );
+  }
+}

lib/pages/home/accounts/search.dart

@@ -200,7 +200,14 @@ class _AccountSearchState extends State<AccountSearch> {
                 ? PagedListView<int, AccountRead>(
                     pagingController: _pagingController,
                     builderDelegate: PagedChildBuilderDelegate<AccountRead>(
-                      itemBuilder: accountRowBuilder,
+                      itemBuilder:
+                          (BuildContext context, AccountRead item, int index) =>
+                              accountRowBuilder(
+                        context,
+                        item,
+                        index,
+                        _pagingController,
+                      ),
                     ),
                   )
                 : const SizedBox.expand(),

lib/pages/home/main.dart

@@ -26,6 +26,7 @@ import 'package:waterflyiii/pages/home/main_charts/lastdays.dart';
 import 'package:waterflyiii/pages/home/main_charts/netearnings.dart';
 import 'package:waterflyiii/pages/home/main_charts/networth.dart';
 import 'package:waterflyiii/pages/home/main_charts/summary.dart';
+import 'package:waterflyiii/stock.dart';
 import 'package:waterflyiii/timezonehandler.dart';
 import 'package:waterflyiii/widgets/charts.dart';
 
@@ -53,17 +54,19 @@ class _HomeMainState extends State<HomeMain>
   List<ChartDataSet> overviewChartData = <ChartDataSet>[];
   final List<InsightGroupEntry> catChartData = <InsightGroupEntry>[];
   final Map<String, Budget> budgetInfos = <String, Budget>{};
+  late TransStock _stock;
 
   @override
   void initState() {
     super.initState();
 
-    context.read<FireflyService>().transStock!.addListener(_refreshStats);
+    _stock = context.read<FireflyService>().transStock!;
+    _stock.addListener(_refreshStats);
   }
 
   @override
   void dispose() {
-    context.read<FireflyService>().transStock!.removeListener(_refreshStats);
+    _stock.removeListener(_refreshStats);
 
     super.dispose();
   }