Merge pull request #120 from Onix-Systems/feature/class-name-validation

feat(Validation): Add class name validation

Author: cozvtieg9

lib/app/app_consts.dart

@@ -46,5 +46,7 @@ class AppConsts {
 
   static RegExp digitsAndLatinWithSpaceOnly = RegExp(r'[a-zA-Z\d ]');
 
+  static RegExp digitsAndLatinLetters = RegExp('[a-zA-Z0-9]');
+
   const AppConsts._();
 }

lib/app/localization/generated/intl/messages_en.dart

@@ -215,6 +215,8 @@ class MessageLookup extends MessageLookupByLibrary {
             "Screen with such name already exists"),
         "screenAlreadyExistsTitle":
             MessageLookupByLibrary.simpleMessage("Already exists"),
+        "screenClassNameHelperText": MessageLookupByLibrary.simpleMessage(
+            "The first character in the class name must be a letter"),
         "screenName": MessageLookupByLibrary.simpleMessage("Screen name"),
         "screens": MessageLookupByLibrary.simpleMessage("Screens"),
         "selectProjectFolder":

lib/app/localization/generated/intl/messages_uk.dart

@@ -221,6 +221,8 @@ class MessageLookup extends MessageLookupByLibrary {
             "Екран з такою назвою вже існує"),
         "screenAlreadyExistsTitle":
             MessageLookupByLibrary.simpleMessage("Вже є"),
+        "screenClassNameHelperText": MessageLookupByLibrary.simpleMessage(
+            "Перший символ у назві класу має бути літерою"),
         "screenName": MessageLookupByLibrary.simpleMessage("Назва екрану"),
         "screens": MessageLookupByLibrary.simpleMessage("Екрани"),
         "selectProjectFolder":

lib/app/localization/generated/l10n.dart

@@ -148,5 +148,6 @@
   "projectNameHelperText": "Project name must be fully lowercase, with underscores to separate words. Also make sure that the name is a valid Dart identifier - it does not start with numbers, is not a reserved word, and does not start or end with an underscore.",
   "organizationNameHelperText": "A digit cannot be the first character in a package segment. The package must have at least one '.’ separator. Multiple '.' characters are not allowed in a row.",
   "changeStateManagerError": "Change state manager warning",
-  "changeStateManagerErrorContent": "Changing state manager will change ... to {variant}. Are you sure you want to continue?"
+  "changeStateManagerErrorContent": "Changing state manager will change ... to {variant}. Are you sure you want to continue?",
+  "screenClassNameHelperText": "The first character in the class name must be a letter"
 }

lib/app/localization/l10n/intl_en.arb

@@ -148,5 +148,6 @@
   "projectNameHelperText": "Назва проєкту має бути повністю маленькими літерами, з підкресленнями, щоб відокремити слова. Також переконайтеся, що назва є дійсним ідентифікатором Dart - вона не починається з цифр, не є зарезервованим словом, не починається і не закінчується символом підкреслення",
   "organizationNameHelperText": "Цифра не може бути першим символом у сегменті пакету. У пакеті повинен бути хоча б один роздільник '.'. Кілька символів '.' підряд не допускаються.",
   "changeStateManagerError": "Попередження зміни стейт менеджера",
-  "changeStateManagerErrorContent": "Зміна стейт мененджера змінить ... на {variant}. Дійсно бажаєте продовжити?"
+  "changeStateManagerErrorContent": "Зміна стейт мененджера змінить ... на {variant}. Дійсно бажаєте продовжити?",
+  "screenClassNameHelperText": "Перший символ у назві класу має бути літерою"
 }

lib/app/localization/l10n/intl_uk.arb

@@ -0,0 +1,17 @@
+import 'package:flutter/services.dart';
+
+class FirstCharacterNotDigitFormatter extends TextInputFormatter {
+  const FirstCharacterNotDigitFormatter();
+
+  @override
+  TextEditingValue formatEditUpdate(
+    TextEditingValue oldValue,
+    TextEditingValue newValue,
+  ) {
+    if (newValue.text.isNotEmpty &&
+        newValue.text[0].contains(RegExp('[0-9]'))) {
+      return oldValue;
+    }
+    return newValue;
+  }
+}

lib/app/util/formatters/first_character_is_not_digit_formatter.dart

@@ -1,11 +1,14 @@
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/services.dart';
 import 'package:gap/gap.dart';
+import 'package:onix_flutter_bricks/app/app_consts.dart';
 import 'package:onix_flutter_bricks/app/localization/generated/l10n.dart';
+import 'package:onix_flutter_bricks/app/util/formatters/first_character_is_not_digit_formatter.dart';
 import 'package:onix_flutter_bricks/domain/entity/screen/screen.dart';
 import 'package:onix_flutter_bricks/domain/entity/state_management/state_management_variant.dart';
 import 'package:onix_flutter_bricks/presentation/style/theme/theme_extension/ext.dart';
 import 'package:onix_flutter_bricks/presentation/widget/inputs/labeled_checkbox.dart';
+import 'package:onix_flutter_bricks/presentation/widget/tooltip_wrapper.dart';
 import 'package:onix_flutter_bricks/util/extension/swagger_extensions.dart';
 import 'package:recase/recase.dart';
 
@@ -77,20 +80,33 @@ class _AddScreenDialogState extends State<AddScreenDialog> {
         content: Column(
           crossAxisAlignment: CrossAxisAlignment.start,
           children: [
-            CupertinoTextField(
-              controller: _screenNameController,
-              focusNode: _textFieldFocusNode,
-              style: context.appTextStyles.fs18,
-              onTap: () {
-                setState(() {
-                  _currentFocusNode = _textFieldFocusNode;
-                  _textFieldFocusNode.requestFocus();
-                });
-              },
-              onSubmitted: (_) => _onOk(context),
-              placeholder: S.of(context).screenName,
-              inputFormatters: [
-                FilteringTextInputFormatter.allow(RegExp('[a-zA-Z0-9]')),
+            Row(
+              children: [
+                Expanded(
+                  child: CupertinoTextField(
+                    controller: _screenNameController,
+                    focusNode: _textFieldFocusNode,
+                    style: context.appTextStyles.fs18,
+                    onTap: () {
+                      setState(() {
+                        _currentFocusNode = _textFieldFocusNode;
+                        _textFieldFocusNode.requestFocus();
+                      });
+                    },
+                    onSubmitted: (_) => _onOk(context),
+                    placeholder: S.of(context).screenName,
+                    inputFormatters: [
+                      const FirstCharacterNotDigitFormatter(),
+                      FilteringTextInputFormatter.allow(
+                        AppConsts.digitsAndLatinLetters,
+                      ),
+                    ],
+                  ),
+                ),
+                const SizedBox(width: 12),
+                TooltipWrapper(
+                  message: S.of(context).screenClassNameHelperText,
+                ),
               ],
             ),
             const Gap(15),
@@ -133,18 +149,20 @@ class _AddScreenDialogState extends State<AddScreenDialog> {
     );
   }
 
-  Future<void> _onOk(BuildContext context) async {
+  void _onOk(BuildContext context) {
     if (_screenNameController.text.isNotEmpty) {
       var screenName = _screenNameController.text.pascalCase;
 
       while (screenName.endsWith('Screen')) {
         screenName = screenName.replaceLast('Screen', '');
       }
 
-      if (widget.screen != null) {
-        widget.screen!.name = screenName;
-        widget.screen!.stateVariant = _stateManagement;
-        Navigator.pop(context, widget.screen);
+      final screen = widget.screen;
+      if (screen != null) {
+        screen
+          ..name = screenName
+          ..stateVariant = _stateManagement;
+        Navigator.pop(context, screen);
       } else {
         Navigator.pop(
           context,