Project

General

Profile

Actions

Bug #21255

closed

Misleading notice when creating or deleting ENABLE_INSTALL_TOOL file

Added by Marc Wöhlken about 15 years ago. Updated almost 15 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2009-10-12
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

This bug applies to TYPO3 4.3-dev (not selectable in bugtracker):

Creating or deleting the install tool enable file (ENABLE_INSTALL_TOOL) in module "User Settings" on tab "Admin functions" displays the following misleading message:

"New password: Password was NOT updated because you didn't enter the same password twice"

(issue imported from #M12210)


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #21564: User settings module reports changed password and settings if there was no change at allClosedOliver Hader2009-11-16

Actions
Related to TYPO3 Core - Bug #21356: Misleading flash message when creation/deletion install tool file (mod setup)ClosedOliver Hader2009-10-25

Actions
Actions #1

Updated by Chris topher about 15 years ago

I also noticed that.
I think the reason might be, that I entered the password-data in my browser to fill out the fields automatically.
Then, when I open the module, the first password-field is filled, but the second is not.

Is that also the reason for you?

Actions #2

Updated by Marc Wöhlken about 15 years ago

Good point!

Clearing the password field manually does indeed solve the problem.

Maybe renaming the password field in module "User settings" could solve this issue permanently?

Actions #3

Updated by Marc Wöhlken about 15 years ago

BTW: Why does the "Create enable install tool" button to the same HTML-form that holds the user data?

Isn't saving / resetting user data a complete different task?

Actions #4

Updated by Steffen Kamper about 15 years ago

i don't see a problem with this message. The autofill is done by browser, not by code. When you don't use the autofill (eg IE) the message doesn't appear.

Cleaning from code isn't possible because browser fills the field after render.

Actions #5

Updated by Chris topher about 15 years ago

The problem is, that the enable-button actually saves the content of the form!
But that is what the save-button should do!
The enable-button should NOT save anything. This is really confusing!

Actions #6

Updated by Marc Wöhlken about 15 years ago

In my eyes the problem is that two actions are commited when trying to create the ENABLE_INSTALL_TOOL file:
1. ENABLE_INSTALL_TOOL (this is expected)
2. All user data is submitted and saved (which is NOT expected)

A solution might be to separate these actions.

Probably the button to create the ENABLE_INSTALL_TOOL file has to be moved out of the user data HTML form.

Actions #7

Updated by Chris topher about 15 years ago

The button was moved to this tab newly. But if I remember correctly noone considered this problem... The idea of moving it back sounds good.

Actions #8

Updated by Marc Wöhlken about 15 years ago

I still don't like the button to trigger to distinct actions but one simple solution not to break the current logic would be to disable autocomplete for the password field.

Simply add autocomplete="off" to the corresponding input and we're done!

This is compliant to XHTML 1.0 transitional, see http://aktuell.de.selfhtml.org/artikel/html/autocomplete/

Actions #9

Updated by Chris topher about 15 years ago

#21271 is solved, but the problem still remains:

The button to create the enable file saves the contents of the tabs. This should not be the case.
And - if possible - it also should not reset changed values which are still unsaved.

Can you create a patch fixing this wrong behaviour?

Actions #10

Updated by Oliver Hader almost 15 years ago

This was fixed with TYPO3 4.3-RC1

Actions

Also available in: Atom PDF