? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
8 Nov 2014

Executive summary

This PR converts the form validation on com_finder to use plain jquery (no mootools call on every form

avatar dgt41 dgt41 - open - 8 Nov 2014
avatar jissues-bot jissues-bot - change - 8 Nov 2014
The description was changed
Labels Added: ?
avatar brianteeman brianteeman - change - 8 Nov 2014
Category JavaScript
avatar smanzi
smanzi - comment - 12 Nov 2014

@dgt41 It seems to work, but I'm unable (both with/without this PR) to open any children in Content Maps. Link is the same for every "Title" (Type): /administrator/index.php?option=com_finder&view=maps# Searched in Joomla Issues but can't find anything about this. New bug?

I must say that I'm not a big fan of com_finder as it has serious flaws in multilingual environments and I didn't use it since long...

avatar dgt41
dgt41 - comment - 12 Nov 2014

@smanzi I don’t get any errors here:
screen shot 2014-11-12 at 4 05 37
Are you still on 3.3.6? Try to get the files of staging repo in your local environment, many things have been fixed the last weeks

avatar smanzi
smanzi - comment - 12 Nov 2014

@dgt41 Yeah, no error me too, but I see you have not indexed your content: try to index it, then go back to the "Content Maps" and try to open any map that has has children...

avatar smanzi
smanzi - comment - 12 Nov 2014

@dgt41 Just a silly question (I probably should know, but as a matter of fact I don't): I have my GitHub repo locally, but... how do I "Install" it to my local server where I have 3.3.6 installed? Just copy different files? No DB work to do?

avatar dgt41
dgt41 - comment - 12 Nov 2014

Just overwrite the files, but be careful you might lose installed staff (overwriting a folder in a mac actually replaces the folder

avatar smanzi
smanzi - comment - 12 Nov 2014

@No problem syncing under Windows: wonderful "Beyond compare" application! :-)
Now that you have indexed, try clicking Author or Type...

avatar dgt41
dgt41 - comment - 12 Nov 2014

screen shot 2014-11-12 at 4 29 13
Works!

avatar dgt41
dgt41 - comment - 12 Nov 2014

@smanzi Try to go: http://localhost/administrator/index.php?option=com_installer&view=database and fix the db!

avatar smanzi
smanzi - comment - 12 Nov 2014

@dgt41 crap! not here... I'll try syncing my installation with staging... :-( tnxs!

What form should I test here, in com_finder?

avatar dgt41
dgt41 - comment - 12 Nov 2014

In indexed content try: index, purge, statistics
In content maps try to publish unpublish something
and in search filter try to create a new one

avatar smanzi
smanzi - comment - 12 Nov 2014

@dgt41 New search filter without title gives warning instead of error, otherwise:

@test success

avatar dgt41
dgt41 - comment - 12 Nov 2014

@smanzi if you want to make a PR for that will be nice. hint: try to add something like

if (jQuery(‘#jform_type’) == ‘’)
{
alert(“Error message");
}
avatar smanzi
smanzi - comment - 12 Nov 2014

@dgt41 OMG, I'll have to fight with JS again! Will try... :-)
I'll wait until your PR has been merged, to not mix things up...

avatar dgt41
dgt41 - comment - 12 Nov 2014

@smanzi That was a bad comment. This was supposed to go in com_menus…

avatar smanzi
smanzi - comment - 12 Nov 2014

Actually here in com_finder for a missing field we have this:
warning
while in all other components we have this:
error
IMHO an error is better than a warning, but the associated text used in com_finder is more approriate, and I think it will be correct to display the "Error:" title: is this something under the control of your form validator or is it responsibility of each component?

avatar anibalsanchez
anibalsanchez - comment - 13 Nov 2014

@test success

  1. In indexed content try: index, purge, statistics, OK
  2. In content maps try to publish unpublish something, OK
  3. and in search filter try to create a new one, OK

In the search filter creation, there is a console Javascript error... it may be unrelated with the validator:

sliderfilter.js

ReferenceError: Class is not defined
FinderFilter = new Class({

avatar dgt41
dgt41 - comment - 13 Nov 2014

@anibalsanchez I am afraid the same error existed before the new code, I tried to sort it out but with no success. This was already broken!

avatar chrisdavenport
chrisdavenport - comment - 13 Nov 2014

There is a pre-existing JavaScript error on this screen, reported here: #5018
A fix would be very welcome.

avatar dgt41
dgt41 - comment - 19 Nov 2014

@roland-d this one got 2 good tests as well. As for the search filter creation bug is solved here: #5099 But on that one the PLT has to decide which of the two proposed solutions will finally merge to the codebase. I think @chrisdavenport is following that bug

avatar roland-d roland-d - alter_testresult - 19 Nov 2014 - smanzi: Tested successfully
avatar roland-d roland-d - alter_testresult - 19 Nov 2014 - anibalsanchez: Tested successfully
avatar roland-d roland-d - test_item - 19 Nov 2014 - Tested successfully
avatar roland-d roland-d - change - 19 Nov 2014
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 19 Nov 2014

Moving to RTC as we have 3 successful tests.

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

avatar brianteeman brianteeman - change - 29 Nov 2014
Labels Added: ?
avatar phproberto phproberto - close - 29 Nov 2014
avatar zero-24 zero-24 - close - 29 Nov 2014
avatar phproberto phproberto - change - 29 Nov 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-11-29 22:38:00
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment