[FIX] signup_recaptcha - override AuthSignupHome

from auth_remove_space_in_signup_and_login

.editorconfig

@@ -1,20 +0,0 @@
-# Configuration for known file extensions
-[*.{css,js,json,less,md,py,rst,sass,scss,xml,yaml,yml}]
-charset = utf-8
-end_of_line = lf
-indent_size = 4
-indent_style = space
-insert_final_newline = true
-trim_trailing_whitespace = true
-
-[*.{json,yml,yaml,rst,md}]
-indent_size = 2
-
-# Do not configure editor for libs and autogenerated content
-[{*/static/{lib,src/lib}/**,*/static/description/index.html,*/readme/../README.rst}]
-charset = unset
-end_of_line = unset
-indent_size = unset
-indent_style = unset
-insert_final_newline = false
-trim_trailing_whitespace = false

.eslintrc.yml

@@ -1,188 +0,0 @@
-env:
-  browser: true
-  es6: true
-
-# See https://github.com/OCA/odoo-community.org/issues/37#issuecomment-470686449
-parserOptions:
-  ecmaVersion: 2019
-
-overrides:
-  - files:
-      - "**/*.esm.js"
-    parserOptions:
-      sourceType: module
-
-# Globals available in Odoo that shouldn't produce errorings
-globals:
-  _: readonly
-  $: readonly
-  fuzzy: readonly
-  jQuery: readonly
-  moment: readonly
-  odoo: readonly
-  openerp: readonly
-  owl: readonly
-  luxon: readonly
-
-# Styling is handled by Prettier, so we only need to enable AST rules;
-# see https://github.com/OCA/maintainer-quality-tools/pull/618#issuecomment-558576890
-rules:
-  accessor-pairs: warn
-  array-callback-return: warn
-  callback-return: warn
-  capitalized-comments:
-    - warn
-    - always
-    - ignoreConsecutiveComments: true
-      ignoreInlineComments: true
-  complexity:
-    - warn
-    - 15
-  constructor-super: warn
-  dot-notation: warn
-  eqeqeq: warn
-  global-require: warn
-  handle-callback-err: warn
-  id-blacklist: warn
-  id-match: warn
-  init-declarations: error
-  max-depth: warn
-  max-nested-callbacks: warn
-  max-statements-per-line: warn
-  no-alert: warn
-  no-array-constructor: warn
-  no-caller: warn
-  no-case-declarations: warn
-  no-class-assign: warn
-  no-cond-assign: error
-  no-const-assign: error
-  no-constant-condition: warn
-  no-control-regex: warn
-  no-debugger: error
-  no-delete-var: warn
-  no-div-regex: warn
-  no-dupe-args: error
-  no-dupe-class-members: error
-  no-dupe-keys: error
-  no-duplicate-case: error
-  no-duplicate-imports: error
-  no-else-return: warn
-  no-empty-character-class: warn
-  no-empty-function: error
-  no-empty-pattern: error
-  no-empty: warn
-  no-eq-null: error
-  no-eval: error
-  no-ex-assign: error
-  no-extend-native: warn
-  no-extra-bind: warn
-  no-extra-boolean-cast: warn
-  no-extra-label: warn
-  no-fallthrough: warn
-  no-func-assign: error
-  no-global-assign: error
-  no-implicit-coercion:
-    - warn
-    - allow: ["~"]
-  no-implicit-globals: warn
-  no-implied-eval: warn
-  no-inline-comments: warn
-  no-inner-declarations: warn
-  no-invalid-regexp: warn
-  no-irregular-whitespace: warn
-  no-iterator: warn
-  no-label-var: warn
-  no-labels: warn
-  no-lone-blocks: warn
-  no-lonely-if: error
-  no-mixed-requires: error
-  no-multi-str: warn
-  no-native-reassign: error
-  no-negated-condition: warn
-  no-negated-in-lhs: error
-  no-new-func: warn
-  no-new-object: warn
-  no-new-require: warn
-  no-new-symbol: warn
-  no-new-wrappers: warn
-  no-new: warn
-  no-obj-calls: warn
-  no-octal-escape: warn
-  no-octal: warn
-  no-param-reassign: warn
-  no-path-concat: warn
-  no-process-env: warn
-  no-process-exit: warn
-  no-proto: warn
-  no-prototype-builtins: warn
-  no-redeclare: warn
-  no-regex-spaces: warn
-  no-restricted-globals: warn
-  no-restricted-imports: warn
-  no-restricted-modules: warn
-  no-restricted-syntax: warn
-  no-return-assign: error
-  no-script-url: warn
-  no-self-assign: warn
-  no-self-compare: warn
-  no-sequences: warn
-  no-shadow-restricted-names: warn
-  no-shadow: warn
-  no-sparse-arrays: warn
-  no-sync: warn
-  no-this-before-super: warn
-  no-throw-literal: warn
-  no-undef-init: warn
-  no-undef: error
-  no-unmodified-loop-condition: warn
-  no-unneeded-ternary: error
-  no-unreachable: error
-  no-unsafe-finally: error
-  no-unused-expressions: error
-  no-unused-labels: error
-  no-unused-vars: error
-  no-use-before-define: error
-  no-useless-call: warn
-  no-useless-computed-key: warn
-  no-useless-concat: warn
-  no-useless-constructor: warn
-  no-useless-escape: warn
-  no-useless-rename: warn
-  no-void: warn
-  no-with: warn
-  operator-assignment: [error, always]
-  prefer-const: warn
-  radix: warn
-  require-yield: warn
-  sort-imports: warn
-  spaced-comment: [error, always]
-  strict: [error, function]
-  use-isnan: error
-  valid-jsdoc:
-    - warn
-    - prefer:
-        arg: param
-        argument: param
-        augments: extends
-        constructor: class
-        exception: throws
-        func: function
-        method: function
-        prop: property
-        return: returns
-        virtual: abstract
-        yield: yields
-      preferType:
-        array: Array
-        bool: Boolean
-        boolean: Boolean
-        number: Number
-        object: Object
-        str: String
-        string: String
-      requireParamDescription: false
-      requireReturn: false
-      requireReturnDescription: false
-      requireReturnType: false
-  valid-typeof: warn
-  yoda: warn

.gitea/workflows/pre-commit.yml

@@ -1,42 +0,0 @@
-name: pre-commit
-
-on:
-  pull_request:
-    branches:
-      - "16.0*"
-
-jobs:
-  pre-commit:
-    runs-on: ubuntu-22.04
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-python@v5
-        with:
-          python-version: "3.11"
-      - name: Get python version
-        run: echo "PY=$(python -VV | sha256sum | cut -d' ' -f1)" >> $GITHUB_ENV
-      # - uses: actions/cache@v4
-      #   with:
-      #     path: ~/.cache/pre-commit
-      #     key: pre-commit|${{ env.PY }}|${{ hashFiles('.pre-commit-config.yaml') }}
-      - name: Install pre-commit
-        run: pip install pre-commit
-      - name: Run pre-commit
-        run: pre-commit run --all-files --show-diff-on-failure --color=always
-        env:
-          # Consider valid a PR that changes README fragments but doesn't
-          # change the README.rst file itself. It's not really a problem
-          # because the bot will update it anyway after merge. This way, we
-          # lower the barrier for functional contributors that want to fix the
-          # readme fragments, while still letting developers get README
-          # auto-generated (which also helps functionals when using runboat).
-          # DOCS https://pre-commit.com/#temporarily-disabling-hooks
-          SKIP: oca-gen-addon-readme
-      - name: Check that all files generated by pre-commit are in git
-        run: |
-          newfiles="$(git ls-files --others --exclude-from=.gitignore)"
-          if [ "$newfiles" != "" ] ; then
-              echo "Please check-in the following files:"
-              echo "$newfiles"
-              exit 1
-          fi

.pre-commit-config.yaml

@@ -1,150 +0,0 @@
-exclude: |
-  (?x)
-  # NOT INSTALLABLE ADDONS
-  # END NOT INSTALLABLE ADDONS
-  # Files and folders generated by bots, to avoid loops
-  ^setup/|/static/description/index\.html$|
-  # We don't want to mess with tool-generated files
-  .svg$|/tests/([^/]+/)?cassettes/|^.copier-answers.yml$|^.github/|^eslint.config.cjs|^prettier.config.cjs|
-  # Maybe reactivate this when all README files include prettier ignore tags?
-  ^README\.md$|
-  # Library files can have extraneous formatting (even minimized)
-  /static/(src/)?lib/|
-  # Repos using Sphinx to generate docs don't need prettying
-  ^docs/_templates/.*\.html$|
-  # Don't bother non-technical authors with formatting issues in docs
-  readme/.*\.(rst|md)$|
-  # Ignore build and dist directories in addons
-  /build/|/dist/|
-  # Ignore test files in addons
-  /tests/samples/.*|
-  # You don't usually want a bot to modify your legal texts
-  (LICENSE.*|COPYING.*)
-default_language_version:
-  python: python3
-  node: "16.17.0"
-repos:
-  - repo: local
-    hooks:
-      # These files are most likely copier diff rejection junks; if found,
-      # review them manually, fix the problem (if needed) and remove them
-      - id: forbidden-files
-        name: forbidden files
-        entry: found forbidden files; remove them
-        language: fail
-        files: "\\.rej$"
-      - id: en-po-files
-        name: en.po files cannot exist
-        entry: found a en.po file
-        language: fail
-        files: '[a-zA-Z0-9_]*/i18n/en\.po$'
-  - repo: https://github.com/oca/maintainer-tools
-    rev: f9b919b9868143135a9c9cb03021089cabba8223
-    hooks:
-      # update the NOT INSTALLABLE ADDONS section above
-      - id: oca-update-pre-commit-excluded-addons
-      - id: oca-fix-manifest-website
-        entry:
-          bash -c 'oca-fix-manifest-website "https://git.elabore.coop/elabore/$(basename
-          $(git rev-parse --show-toplevel))"'
-      - id: oca-gen-addon-readme
-        entry:
-          bash -c 'oca-gen-addon-readme 
-          --addons-dir=. 
-          --branch=$(git symbolic-ref
-          refs/remotes/origin/HEAD | sed "s@^refs/remotes/origin/@@")
-          --repo-name=$(basename $(git rev-parse --show-toplevel))
-          --org-name="Elabore"
-          --if-source-changed --keep-source-digest'
-
-  - repo: https://github.com/OCA/odoo-pre-commit-hooks
-    rev: v0.1.4
-    hooks:
-      - id: oca-checks-odoo-module
-      - id: oca-checks-po
-        args:
-          - --disable=po-pretty-format
-  - repo: local
-    hooks:
-      - id: prettier
-        name: prettier (with plugin-xml)
-        entry: prettier
-        args:
-          - --write
-          - --list-different
-          - --ignore-unknown
-        types: [text]
-        files: \.(css|htm|html|js|json|jsx|less|md|scss|toml|ts|xml|yaml|yml)$
-        language: node
-        additional_dependencies:
-          - "prettier@2.7.1"
-          - "@prettier/plugin-xml@2.2.0"
-  - repo: local
-    hooks:
-      - id: eslint
-        name: eslint
-        entry: eslint
-        args:
-          - --color
-          - --fix
-        verbose: true
-        types: [javascript]
-        language: node
-        additional_dependencies:
-          - "eslint@8.24.0"
-          - "eslint-plugin-jsdoc@"
-  - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v5.0.0
-    hooks:
-      - id: trailing-whitespace
-        # exclude autogenerated files
-        exclude: /README\.rst$|\.pot?$
-      - id: end-of-file-fixer
-        # exclude autogenerated files
-        exclude: /README\.rst$|\.pot?$
-      - id: debug-statements
-      - id: fix-encoding-pragma
-        args: ["--remove"]
-      - id: check-case-conflict
-      - id: check-docstring-first
-      - id: check-executables-have-shebangs
-      - id: check-merge-conflict
-        # exclude files where underlines are not distinguishable from merge conflicts
-        exclude: /README\.rst$|^docs/.*\.rst$
-      - id: check-symlinks
-      - id: check-xml
-      - id: mixed-line-ending
-        args: ["--fix=lf"]
-  - repo: https://github.com/PyCQA/docformatter
-    rev: v1.7.7
-    hooks:
-      - id: docformatter
-        args: [
-            "--in-place", # modify the files
-            "--recursive", # run on all the files
-            "--wrap-summaries",
-            "88", # max length of 1st line
-            "--wrap-descriptions",
-            "88", # max length of other lines
-            "--pre-summary-newline", # new line before a long summary
-            "--make-summary-multi-line", # force summary on multilines
-          ]
-        additional_dependencies: ["tomli"] # if Python <3.11
-  - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.12.0
-    hooks:
-      - id: ruff
-        args: [--fix, --exit-non-zero-on-fix]
-      - id: ruff-format
-  - repo: https://github.com/OCA/pylint-odoo
-    rev: v9.1.3
-    hooks:
-      - id: pylint_odoo
-        name: pylint with optional checks
-        args:
-          - --rcfile=.pylintrc
-          - --exit-zero
-        verbose: true
-      - id: pylint_odoo
-        args:
-          - --rcfile=.pylintrc-mandatory

.prettierrc.yml

@@ -1,8 +0,0 @@
-# Defaults for all prettier-supported languages.
-# Prettier will complete this with settings from .editorconfig file.
-bracketSpacing: false
-printWidth: 88
-proseWrap: always
-semi: true
-trailingComma: "es5"
-xmlWhitespaceSensitivity: "strict"

.pylintrc

@@ -1,123 +0,0 @@
-
-
-[MASTER]
-load-plugins=pylint_odoo
-score=n
-
-[ODOOLINT]
-readme-template-url="https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst"
-manifest-required-authors=Elabore
-manifest-required-keys=license
-manifest-deprecated-keys=description,active
-license-allowed=AGPL-3,GPL-2,GPL-2 or any later version,GPL-3,GPL-3 or any later version,LGPL-3
-valid-odoo-versions=16.0
-
-[MESSAGES CONTROL]
-disable=all
-
-# This .pylintrc contains optional AND mandatory checks and is meant to be
-# loaded in an IDE to have it check everything, in the hope this will make
-# optional checks more visible to contributors who otherwise never look at a
-# green travis to see optional checks that failed.
-# .pylintrc-mandatory containing only mandatory checks is used the pre-commit
-# config as a blocking check.
-
-enable=anomalous-backslash-in-string,
-    api-one-deprecated,
-    api-one-multi-together,
-    assignment-from-none,
-    attribute-deprecated,
-    class-camelcase,
-    dangerous-default-value,
-    dangerous-view-replace-wo-priority,
-    development-status-allowed,
-    duplicate-id-csv,
-    duplicate-key,
-    duplicate-xml-fields,
-    duplicate-xml-record-id,
-    eval-referenced,
-    eval-used,
-    incoherent-interpreter-exec-perm,
-    license-allowed,
-    manifest-author-string,
-    manifest-deprecated-key,
-    manifest-required-author,
-    manifest-required-key,
-    manifest-version-format,
-    method-compute,
-    method-inverse,
-    method-required-super,
-    method-search,
-    openerp-exception-warning,
-    pointless-statement,
-    pointless-string-statement,
-    print-used,
-    redundant-keyword-arg,
-    redundant-modulename-xml,
-    reimported,
-    relative-import,
-    return-in-init,
-    rst-syntax-error,
-    sql-injection,
-    too-few-format-args,
-    translation-field,
-    translation-required,
-    unreachable,
-    use-vim-comment,
-    wrong-tabs-instead-of-spaces,
-    xml-syntax-error,
-    attribute-string-redundant,
-    character-not-valid-in-resource-link,
-    consider-merging-classes-inherited,
-    context-overridden,
-    create-user-wo-reset-password,
-    dangerous-filter-wo-user,
-    dangerous-qweb-replace-wo-priority,
-    deprecated-data-xml-node,
-    deprecated-openerp-xml-node,
-    duplicate-po-message-definition,
-    except-pass,
-    file-not-used,
-    invalid-commit,
-    manifest-maintainers-list,
-    missing-newline-extrafiles,
-    missing-readme,
-    missing-return,
-    odoo-addons-relative-import,
-    old-api7-method-defined,
-    po-msgstr-variables,
-    po-syntax-error,
-    renamed-field-parameter,
-    resource-not-exist,
-    str-format-used,
-    test-folder-imported,
-    translation-contains-variable,
-    translation-positional-used,
-    unnecessary-utf8-coding-comment,
-    website-manifest-key-not-valid-uri,
-    xml-attribute-translatable,
-    xml-deprecated-qweb-directive,
-    xml-deprecated-tree-attribute,
-    external-request-timeout,
-    # messages that do not cause the lint step to fail
-    consider-merging-classes-inherited,
-    create-user-wo-reset-password,
-    dangerous-filter-wo-user,
-    deprecated-module,
-    file-not-used,
-    invalid-commit,
-    missing-manifest-dependency,
-    missing-newline-extrafiles,
-    missing-readme,
-    no-utf8-coding-comment,
-    odoo-addons-relative-import,
-    old-api7-method-defined,
-    redefined-builtin,
-    too-complex,
-    unnecessary-utf8-coding-comment
-
-
-[REPORTS]
-msg-template={path}:{line}: [{msg_id}({symbol}), {obj}] {msg}
-output-format=colorized
-reports=no

.pylintrc-mandatory

@@ -1,98 +0,0 @@
-
-[MASTER]
-load-plugins=pylint_odoo
-score=n
-
-[ODOOLINT]
-readme-template-url="https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst"
-manifest-required-authors=Elabore
-manifest-required-keys=license
-manifest-deprecated-keys=description,active
-license-allowed=AGPL-3,GPL-2,GPL-2 or any later version,GPL-3,GPL-3 or any later version,LGPL-3
-valid-odoo-versions=16.0
-
-[MESSAGES CONTROL]
-disable=all
-
-enable=anomalous-backslash-in-string,
-    api-one-deprecated,
-    api-one-multi-together,
-    assignment-from-none,
-    attribute-deprecated,
-    class-camelcase,
-    dangerous-default-value,
-    dangerous-view-replace-wo-priority,
-    development-status-allowed,
-    duplicate-id-csv,
-    duplicate-key,
-    duplicate-xml-fields,
-    duplicate-xml-record-id,
-    eval-referenced,
-    eval-used,
-    incoherent-interpreter-exec-perm,
-    license-allowed,
-    manifest-author-string,
-    manifest-deprecated-key,
-    manifest-required-author,
-    manifest-required-key,
-    manifest-version-format,
-    method-compute,
-    method-inverse,
-    method-required-super,
-    method-search,
-    openerp-exception-warning,
-    pointless-statement,
-    pointless-string-statement,
-    print-used,
-    redundant-keyword-arg,
-    redundant-modulename-xml,
-    reimported,
-    relative-import,
-    return-in-init,
-    rst-syntax-error,
-    sql-injection,
-    too-few-format-args,
-    translation-field,
-    translation-required,
-    unreachable,
-    use-vim-comment,
-    wrong-tabs-instead-of-spaces,
-    xml-syntax-error,
-    attribute-string-redundant,
-    character-not-valid-in-resource-link,
-    consider-merging-classes-inherited,
-    context-overridden,
-    create-user-wo-reset-password,
-    dangerous-filter-wo-user,
-    dangerous-qweb-replace-wo-priority,
-    deprecated-data-xml-node,
-    deprecated-openerp-xml-node,
-    duplicate-po-message-definition,
-    except-pass,
-    file-not-used,
-    invalid-commit,
-    manifest-maintainers-list,
-    missing-newline-extrafiles,
-    missing-readme,
-    missing-return,
-    odoo-addons-relative-import,
-    old-api7-method-defined,
-    po-msgstr-variables,
-    po-syntax-error,
-    renamed-field-parameter,
-    resource-not-exist,
-    str-format-used,
-    test-folder-imported,
-    translation-contains-variable,
-    translation-positional-used,
-    unnecessary-utf8-coding-comment,
-    website-manifest-key-not-valid-uri,
-    xml-attribute-translatable,
-    xml-deprecated-qweb-directive,
-    xml-deprecated-tree-attribute,
-    external-request-timeout
-
-[REPORTS]
-msg-template={path}:{line}: [{msg_id}({symbol}), {obj}] {msg}
-output-format=colorized
-reports=no

.ruff.toml

@@ -1,31 +0,0 @@
-
-target-version = "py310"
-fix = true
-
-[lint]
-extend-select = [
-    "B",
-    "C90",
-    "E501",  # line too long (default 88)
-    "I",  # isort
-    "UP",  # pyupgrade
-]
-extend-safe-fixes = ["UP008"]
-exclude = ["setup/*"]
-
-[format]
-exclude = ["setup/*"]
-
-[lint.per-file-ignores]
-"__init__.py" = ["F401", "I001"]  # ignore unused and unsorted imports in __init__.py
-"__manifest__.py" = ["B018"]  # useless expression
-
-[lint.isort]
-section-order = ["future", "standard-library", "third-party", "odoo", "odoo-addons", "first-party", "local-folder"]
-
-[lint.isort.sections]
-"odoo" = ["odoo"]
-"odoo-addons" = ["odoo.addons"]
-
-[lint.mccabe]
-max-complexity = 16

auth_remove_space_in_signup_and_login/README.rst

@@ -18,7 +18,7 @@ None yet.
 Bug Tracker
 ===========
 
-Bugs are tracked on `our issues website <https://github.com/elabore-coop/auth-tools/issues>`_. In case of
+Bugs are tracked on `our issues website <https://github.com/elabore-coop/auth_remove_space_in_login/issues>`_. In case of
 trouble, please check there if your issue has already been
 reported. If you spotted it first, help us smashing it by providing a
 detailed and welcomed feedback.
@@ -30,7 +30,6 @@ Contributors
 ------------
 
 * Boris Gallet - `Email<mailto:boris.gallet@elabore.coop>` - `Github<https://github.com/b0g>`
-* Stéphan Sainléger - `Github<https://github.com/stephansainleger>`
 
 Funders
 -------

auth_remove_space_in_signup_and_login/__manifest__.py

@@ -3,7 +3,7 @@
 
 {
     "name": "auth_remove_space_in_signup_and_login",
-    "version": "16.0.1.0.0",
+    "version": "12.0.1.0.0",
     "author": "Elabore",
     "website": "https://elabore.coop",
     "maintainer": "Boris Gallet",

auth_remove_space_in_signup_and_login/controllers/auth_signup.py

@@ -1,10 +1,13 @@
+# -*- coding: utf-8 -*-
+import logging
+
 from odoo.addons.auth_signup.controllers.main import AuthSignupHome
 from odoo.http import request
 
 
 class AuthSignupHome(AuthSignupHome):
     def get_auth_signup_qcontext(self):
-        qcontext = super().get_auth_signup_qcontext()
+        qcontext = super(AuthSignupHome, self).get_auth_signup_qcontext()
         if request.params.get("email"):
             qcontext["email"] = request.params.get("email").strip()
         if request.params.get("login"):

auth_remove_space_in_signup_and_login/controllers/web_login.py

@@ -1,11 +1,14 @@
+# -*- coding: utf-8 -*-
+import logging
+
+from odoo.addons.web.controllers.main import Home
 from odoo import http
-from odoo.addons.auth_signup.controllers.main import AuthSignupHome as Home
 from odoo.http import request
 
-
 class Home(Home):
-    @http.route()
+    @http.route('/web/login', type='http', auth="none", sitemap=False)
     def web_login(self, *args, **kw):
-        if request.httprequest.method == "POST" and request.params.get("login"):
+        if request.httprequest.method == 'POST':
+            if request.params.get("login"):
                 request.params["login"] = request.params.get("login").strip()
-        return super().web_login(*args, **kw)
+        return super(Home, self).web_login(*args, **kw)

signup_recaptcha/__manifest__.py

@@ -3,7 +3,7 @@
 
 {
     "name": "signup_recaptcha",
-    "version": "16.0.1.0.0",
+    "version": "12.0.1.0.0",
     "author": "Elabore",
     "website": "https://elabore.coop",
     "maintainer": "Boris Gallet",
@@ -11,7 +11,7 @@
     "category": "Tools",
     "summary": "google recaptcha V2 for signup form with server side validation",
     # any module necessary for this one to work correctly
-    "depends": ["base", "portal"],
+    "depends": ["base", "portal", "auth_remove_space_in_signup_and_login"],
     "data": [
         "views/auth_signup.xml",
         "views/res_config.xml",

signup_recaptcha/controllers/main.py

@@ -1,6 +1,10 @@
+# -*- coding: utf-8 -*-
 import logging
 
-from odoo.addons.auth_signup.controllers.main import AuthSignupHome
+
+from odoo.addons.auth_remove_space_in_signup_and_login.controllers.auth_signup import (
+    AuthSignupHome as CustomAuthSignupHome,
+)
 from odoo.addons.auth_signup.models.res_users import SignupError
 from odoo.http import request
 from odoo import _
@@ -9,7 +13,7 @@ from odoo import _
 _logger = logging.getLogger(__name__)
 
 
-class AuthSignupHome(AuthSignupHome):
+class AuthSignupHome(CustomAuthSignupHome):
     def get_auth_signup_qcontext(self):
         """Add recaptcha to the context"""
         qcontext = super(AuthSignupHome, self).get_auth_signup_qcontext()

signup_recaptcha/views/auth_signup.xml

@@ -2,7 +2,7 @@
 <odoo>
     <data>
         <template id="auth_signup_recaptcha" inherit_id="auth_signup.fields" name="Recaptcha for signup">
-        <xpath expr="//div[hasclass('field-confirm_password')]" position="after">
+        <xpath expr="//div[hasclass('form-group','field-confirm_password')]" position="after">
             <script src="https://www.google.com/recaptcha/api.js" />
             <div
             class="g-recaptcha text-xs-center"