NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
10 Jan 2021

Pull Request for Issue #31981 .

Summary of Changes

  • Make password lock and show exclusive
  • Fix upmerge typo issue
  • Replace jquery code with es6 and move to passwordview js file

Testing Instructions

  • Test password fields
  • Test smtp email settings as in #31724

Actual result BEFORE applying this Pull Request

Smtp test mail with password didn't worked
View Password Icon and Modify button was visible at the same time which doesn't work.

Expected result AFTER applying this Pull Request

Test E-Mail settings work and only "Modify" or "Show Password" Icon exists and work

avatar HLeithner HLeithner - open - 10 Jan 2021
avatar HLeithner HLeithner - change - 10 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Jan 2021
Category JavaScript Repository NPM Change Layout Libraries
avatar toivo
toivo - comment - 10 Jan 2021

I have tested this item ? unsuccessfully on f3b6f0e

The Modify button did not work at all.

The Categories of this RP include JavaScript and NPM Change. Does something special need to be done before or after the RP is applied in Joommla! Patch Tester? I do not use NPM.


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

avatar toivo toivo - test_item - 10 Jan 2021 - Tested unsuccessfully
avatar wilsonge
wilsonge - comment - 10 Jan 2021

Yes this one will need npm to test (or one of the prebuilt packs)

avatar toivo
toivo - comment - 11 Jan 2021

@wilsonge where can I download one of the prebuilt packs to test this PR?


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

avatar brianteeman
brianteeman - comment - 11 Jan 2021

@toivo on github
image

avatar gostn
gostn - comment - 11 Jan 2021

@toivo at end of this pr as always on github

image

avatar toivo
toivo - comment - 11 Jan 2021
avatar toivo
toivo - comment - 11 Jan 2021

@brianteeman @gostn - installing that version is far too complicated: "Environment Setup Incomplete
It looks like you are trying to run Joomla! from our git repository. To do so requires you complete a couple of extra steps first."


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

avatar wilsonge
wilsonge - comment - 11 Jan 2021

@toivo you shouldn't get that - the entire point of that binary is that it has all the environment dependencies (npm + composer) installed. I'll look later.

avatar ceford
ceford - comment - 11 Jan 2021

After applying the patch and npm ci I could not get to see the smtp plain text password at all, before or after save. Is that expected? I did not test sending a test email.


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

avatar HLeithner
HLeithner - comment - 11 Jan 2021

After applying the patch and npm ci I could not get to see the smtp plain text password at all, before or after save. Is that expected? I did not test sending a test email.

yes that's expected, it's part of a security patch in j3

avatar HLeithner
HLeithner - comment - 11 Jan 2021

@brianteeman @gostn - installing that version is far too complicated: "Environment Setup Incomplete
It looks like you are trying to run Joomla! from our git repository. To do so requires you complete a couple of extra steps first."

Download https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/31999/downloads/38993/Joomla_4.0.0-beta7-dev+pr.31999-Development-Update_Package.zip and install it as update package to your j4 installation

alternative you can install the full installation package: https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/31999/downloads/38993/Joomla_4.0.0-beta7-dev+pr.31999-Development-Full_Package.zip

5b384f1 11 Jan 2021 avatar HLeithner cs
avatar HLeithner HLeithner - change - 11 Jan 2021
Labels Added: NPM Resource Changed ?
avatar toivo
toivo - comment - 11 Jan 2021

@HLeithner thanks for those links!


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

avatar toivo
toivo - comment - 11 Jan 2021

I have tested this item successfully on 5b384f1

Tested successfully in Beta7-dev installed from Joomla_4.0.0-beta7-dev+pr.31999-Development-Full_Package.zip.

The Modify button works fine. The View Password button is not there but it is not needed, especially if it would change the functionality of the Sent Test Mail button in a way that is not obvious to an average webmaster.


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

avatar toivo toivo - test_item - 11 Jan 2021 - Tested successfully
avatar HLeithner
HLeithner - comment - 11 Jan 2021

I removed the eye button because it would only work if you enter a new password and would confuse people when they see no existing passwords. So the functionality would be very limited anyway. And I think disabling it if not modified and enable on modify could be more unexpected. ymmv

avatar gostn
gostn - comment - 12 Jan 2021

I have tested this item successfully on 5b384f1


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

avatar gostn gostn - test_item - 12 Jan 2021 - Tested successfully
avatar richard67 richard67 - change - 13 Jan 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 13 Jan 2021

RTC


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

avatar richard67 richard67 - change - 13 Jan 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-01-13 13:28:21
Closed_By richard67
Labels Added: ?
avatar richard67 richard67 - close - 13 Jan 2021
avatar richard67 richard67 - merge - 13 Jan 2021
avatar richard67
richard67 - comment - 13 Jan 2021

Thanks!

Add a Comment

Login with GitHub to post a comment