Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.4] WebauthnField missing translation #41723

Merged

Conversation

heelc29
Copy link
Contributor

@heelc29 heelc29 commented Sep 12, 2023

Summary of Changes

add missing translation PLG_SYSTEM_WEBAUTHN_ERR_NOT_DELETED for js function Joomla.Text._()

used here

Joomla.Text._('PLG_SYSTEM_WEBAUTHN_ERR_NOT_DELETED'),

Testing Instructions

add return call to onAjaxWebauthnDelete
image

Actual result BEFORE applying this Pull Request

error message not translated
image

Expected result AFTER applying this Pull Request

error message translated
image

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@ceford
Copy link
Contributor

ceford commented Sep 22, 2023

I have tested this item ✅ successfully on d3bbd25

Tricky to test - but I do see the untranslated string without the patch and the translated string with the patch. However, without without the patch when I click the button on my Key I get this message above the Authenticator name bar:

 Deprecated: Brick\Math\BigInteger implements the Serializable interface, which is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary) in /Users/ceford/Sites/joomla-cms4/libraries/vendor/brick/math/src/BigInteger.php on line 22

I guess that is a separate issue.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41723.

@HLeithner HLeithner added the bug label Sep 24, 2023
@obuisard obuisard changed the base branch from 4.3-dev to 4.4-dev September 24, 2023 14:21
@obuisard obuisard added this to the Joomla! 4.4.0 milestone Sep 24, 2023
@obuisard obuisard changed the title [4.3] WebauthnField missing translation [4.4] WebauthnField missing translation Sep 24, 2023
@obuisard obuisard merged commit 96ebe54 into joomla:4.4-dev Sep 24, 2023
3 checks passed
@obuisard
Copy link
Contributor

Thank you @heelc29 for the PR.

@heelc29 heelc29 deleted the 4.3/webauthnfield/missing-translation branch September 24, 2023 15:31
HLeithner pushed a commit that referenced this pull request Sep 26, 2023
* Added api test for user field group (#41850)

* Added api test for user field group (#41851)

* system test com_config webservices (#41765)

* system test com_config

* id

* divideandrollback

* grr

* rollback

---------

Co-authored-by: Allon Moritz <allon.moritz@digital-peak.com>

* [4.3] WebauthnField missing translation (#41723)

* redo of #41520 (#41856)

* [4.4] add system tests for admin users (group/level) (#41910)

* Use $document variable (#41917)

* redirect (#41912)

* Cypress test for com_privacy admin (#41834)

* Cypress test for com_privacy admin

* Update Consent.cy.js

* [4.4] 500 error when 404 and debug on (#41893)

* fix

* missed new params

* [4] Mismatched tag ids/tag names in articles webservice/api (#41397)

* Mismatched tag ids/tag names in articles webservice/api

* redone

* new function

* use the new helper function

* cs

---------

Co-authored-by: Martin Carl Kopp <6154099+MacJoom@users.noreply.github.com>

* [4.4] Joomla Update Component check package when using Upload & Update (#41757)

* Add checks of uploaded file to com_joomlaupdate

* Use global namespace

* Simplify language strings

* Remove the major version check

* Fix doc block of new method

* Language string improvements - thanks Brian

Co-authored-by: Brian Teeman <brian@teeman.net>

* Only check if zip PHP extension is loaded

* Check manifest XML instead of Version.php

* English improvements

* Remove empty line from language file

Co-authored-by: George Wilson <georgejameswilson@googlemail.com>

* Add alternative method if no zip PHP extension

* Do it as elswehere

* Remove unsupported compression method

* Make sure file is opened as binary file

* Remove special version suffix for pull request patched packages

* Move version check to own method

* Read chunks of max. 1MiB from central directory

* Fix method name and calls

* Fix code from tests with a CLI script

* Add code comments

* Small code simplification

* Fix empy manifest XML file handling

* Handle empty manifest file and start of file

* Fix method description comment

* Close file when throwing exceptions while open

* Fix typo in code comment

* Fix code comment.

Co-authored-by: Quy <quy@nomonkeybiz.com>

* Language string consistency

---------

Co-authored-by: Brian Teeman <brian@teeman.net>
Co-authored-by: George Wilson <georgejameswilson@googlemail.com>
Co-authored-by: Allon Moritz <allon.moritz@digital-peak.com>
Co-authored-by: Quy <quy@nomonkeybiz.com>

* Update template.es6.js (#41909)

Optional chaining fix

* Add build tool to check ruleset.xml for obsolete exclude patterns and optionally fix the file (#40278)

* Add tool to check ruleset.xml for obsolete exclude patterns

* Fix PHPCS

* Add option to fix the XML file

* Rename to check_ruleset_xml.php

* Better help text for default value

* Fix comment, too

* [4][webservice] user patch (#41688)

* [4] webservice com_user patch

PATCH user alters password

* system test

---------

Co-authored-by: Allon Moritz <allon.moritz@digital-peak.com>

* Update tests/System/integration/administrator/components/com_users/Groups.cy.js

Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>

* Update tests/System/integration/administrator/components/com_users/Levels.cy.js

Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>

* Use maximebf/debugbar v1.19.0 (#41931)

* Rebuild composer lock file

* Explanation why we mix UI with API tests

* Rebuild composer lock file because of webauth upgrade

* Fix code style

---------

Co-authored-by: rajputanuj31 <115709571+rajputanuj31@users.noreply.github.com>
Co-authored-by: Nicola Galgano <optimus4joomla@gmail.com>
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
Co-authored-by: andyforrest <46865032+andyforrest@users.noreply.github.com>
Co-authored-by: Denitz <197527+Denitz@users.noreply.github.com>
Co-authored-by: Martin Carl Kopp <6154099+MacJoom@users.noreply.github.com>
Co-authored-by: Brian Teeman <brian@teeman.net>
Co-authored-by: George Wilson <georgejameswilson@googlemail.com>
Co-authored-by: Quy <quy@nomonkeybiz.com>
Co-authored-by: Olivier Buisard <olivier.buisard@simplifyyourweb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants