[MDL-76679] Admin config fields of type password inconsistent behaviour - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 4.0.5
Fix Version/s: 3.11.12, 4.0.6, 4.1.1
Component/s: Administration, Forms Library
Labels:
- ci
- regression
- release_notes
- triaged

Affected Branches:
MOODLE_400_STABLE
Fixed Branches:
MOODLE_311_STABLE, MOODLE_400_STABLE, MOODLE_401_STABLE
Pull from Repository:
https://github.com/paulholden/moodle.git
Pull 4.0 Branch:
~~MDL-76679~~-400
Pull 4.0 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_400_STABLE...paulholden:MDL-76679-400
Pull 4.1 Branch:
~~MDL-76679~~-401
Pull 4.1 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_401_STABLE...paulholden:MDL-76679-401
Pull Main Branch:
~~MDL-76679~~
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...paulholden:MDL-76679
Testing Instructions:
Hide

Log in as admin

Navigate to Security > Site security settings in site administration

Confirm the input field for Cron password for remote access is not initially shown

Press Edit password

Confirm the input field is now shown and can be edited

Enter a value in the field and press enter

Press Reveal

Confirm the password field content is shown
Show
Log in as admin Navigate to Security > Site security settings in site administration Confirm the input field for Cron password for remote access is not initially shown Press Edit password Confirm the input field is now shown and can be edited Enter a value in the field and press enter Press Reveal Confirm the password field content is shown

Description

See screencast

When loading the config field, the input form is initially shown alongside the Edit password link, whereas it should be hidden until that link is pressed

This is a regression from ~~MDL-74814~~, the template changes that were made there also need making to the admin config password field type (ping huongn@moodle.com)

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

(I) Passed -- (311)MDL-76679.mp4
2.68 MB
28/Dec/22 9:01 AM
(I) Passed -- (400)MDL-76679.mp4
2.44 MB
28/Dec/22 9:01 AM
(I) Passed -- (401)MDL-76679.mp4
2.78 MB
28/Dec/22 9:01 AM
(I) Passed -- (Master)MDL-76679.mp4
2.53 MB
28/Dec/22 9:01 AM
Screencast 2022-12-14.gif
49 kB
15/Dec/22 3:09 AM

Issue Links

is a regression caused by

MDL-74814 Some accessibility issues in the edit profile page

Closed

Activity

People

Assignee:: Paul Holden

Reporter:: Paul Holden

Peer reviewer:: Huong Nguyen

Integrator:: Sara Arjona (@sarjona)

Tester:: Kim Jared Lucas

Participants:: CiBoT, Eric Merrill, Huong Nguyen, Kim Jared Lucas, Paul Holden, Sara Arjona (@sarjona)

Votes:: 0 Vote for this issue

Watchers:: 7 Start watching this issue

Dates

Created:: 15/Dec/22 3:06 AM

Updated:: 11/Jan/23 8:21 AM

Resolved:: 29/Dec/22 9:35 PM

Fix Release Date:: 16/Jan/23

Time Tracking

Estimated:

Not Specified

Remaining:

Logged:

1h 19m