Conflicting Files ? ? Success

User tests: Successful: Unsuccessful:

avatar durubayram
durubayram
3 Aug 2018

Pull Request for Issue #21249 .

Summary of Changes

by adding a new error type(COM_CONFIG_ERROR_VALIDATION) in ApplicationController.php file, you're going to be able to see description why form validation failed instead of getting an error which is 'No database selected'

Testing Instructions

Edit the Global configuration and provide an incorrect value for "session lifetime" : 19999
(it is possible to bypass the maximum value of the input field, which is 16383)

Expected result

The configuration is not saved and the user get a error message with the explanation of the issue which is 'Form validation failed. Please check the form.'
ekran goruntusu 83

avatar durubayram durubayram - open - 3 Aug 2018
avatar durubayram durubayram - change - 3 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2018
Category Administration com_config
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 3 Aug 2018 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Aug 2018

I have tested this item ? unsuccessfully on 35e8d5b

19999 is saved and Message shows Constant.

Please ignore Test as for now Front- and Backend-PRs are able to test by Patchtester.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21384.
avatar SharkyKZ
SharkyKZ - comment - 3 Aug 2018

Use error from model:
$this->app->enqueueMessage($model->getError(), 'error');

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 3 Aug 2018 - Not tested
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Aug 2018

I have not tested this item.


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 3 Aug 2018 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Aug 2018

I have tested this item ? unsuccessfully on 35e8d5b

19999 is saved and Message shows Constant.

screen shot 2018-08-03 at 15 02 46


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21384.
avatar Twincarb
Twincarb - comment - 3 Aug 2018

Is the PR missing the string being added to the language file?

avatar wojsmol
wojsmol - comment - 3 Aug 2018

@Twincarb yes

avatar durubayram durubayram - change - 4 Aug 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 4 Aug 2018
Category Administration com_config Administration com_config Language & Strings
avatar durubayram durubayram - change - 4 Aug 2018
Labels Added: ?
avatar wojsmol
wojsmol - comment - 4 Aug 2018
FILE: ...dministrator/components/com_config/Controller/ApplicationController.php
--------------------------------------------------------------------------------
FOUND 2 ERROR(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
 122 | ERROR | Tabs must be used to indent lines; spaces are not allowed
 123 | ERROR | Tabs must be used to indent lines; spaces are not allowed
--------------------------------------------------------------------------------
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY
--------------------------------------------------------------------------------
avatar Quy Quy - change - 4 Aug 2018
Title
it shows that description of error to add new error type
[4.0] it shows that description of error to add new error type
avatar joomla-cms-bot joomla-cms-bot - edited - 4 Aug 2018
avatar joomla-cms-bot joomla-cms-bot - change - 4 Aug 2018
Title
it shows that description of error to add new error type
[4.0] it shows that description of error to add new error type
avatar wilsonge
wilsonge - comment - 6 Aug 2018

Codestyle fixed

avatar brianteeman brianteeman - test_item - 6 Aug 2018 - Tested successfully
avatar brianteeman
brianteeman - comment - 6 Aug 2018

I have tested this item successfully on 1a3c4eb


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 6 Aug 2018 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Aug 2018

I have tested this item ? unsuccessfully on 1a3c4eb

Language String shown correct, but incorrect value for "session lifetime" "19999" is saved

System information

  • Nightly Build 4.0.0-alpha5-dev
  • Multilingual Sample Data (French, German DE, Persian)
  • Template: Cassiopeia
  • macOS Sierra, 10.13.6
  • Firefox 61 (64-bit)

CloudAccess.net

avatar obsidev
obsidev - comment - 6 Aug 2018

Hi,

The "session lifetime" is not saved in the configuration ; it is coming from the session which store the form data to not loose what the user was editing.
So the user know that the configuration is not saved but its form is kept which allow him to fix what is wrong (but he need to know what is wrong)

There is a comment saying that the model should display the error message (which is not the case and we don't know why).

The validate method enqueued all messages for us, so we just need to redirect back.

Letting the model display the errors or pushing the errors from the model is a requirement because the user must know what is going from with the provided data.
The patch fix the issue but it would require further improvements to have a good user experience.

Regards,

avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Apr 2019
Category Administration com_config Language & Strings Administration com_config
avatar rmittl
rmittl - comment - 19 Oct 2019

Using "Blanks" before or after the value is accepted.
No error message - the value after saving is 15.
Example " 18005", "18005 " is value "15" after saving

Proposed solution, trim before check.

avatar vijaykhollam vijaykhollam - test_item - 19 Oct 2019 - Tested unsuccessfully
avatar vijaykhollam
vijaykhollam - comment - 19 Oct 2019

I have tested this item ? unsuccessfully on 88726e0

#
Language String is shown correct, but incorrect value for "session lifetime" "19999" is saved

System information
Joomla! 4.0.0-beta1-dev Development [ Amani ] 17-October-2019 20:21 GMT
Web Server : Apache
WebServer to PHP Interface : cgi-fcgi
PHP 7.3.8
MySQL 5.7.23-cll-lve


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

avatar pravinTek
pravinTek - comment - 19 Oct 2019

Hi,
Now language string showing a correct message and also incorrect value for "session lifetime" "19999" is not storing in config.php
But In the backend UI System -> Global Configuration still showing "19999" this value under "session lifetime".

avatar Quy
Quy - comment - 2 Jan 2020

This is no longer an issue.

21384

avatar Quy Quy - close - 2 Jan 2020
avatar Quy Quy - change - 2 Jan 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-01-02 01:33:16
Closed_By Quy
Labels Added: Conflicting Files
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jan 2020
Category Administration com_config Administration com_config Language & Strings

Add a Comment

Login with GitHub to post a comment