Feeling Lucky
?
[#9078] - Remove the last mootools dependency from com_finder
- Closed
- 10 Feb 2016
- Medium
- Build: staging
- # 9078
- Diff
- nonumber:patch-1
- Pending Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
The handleFailure method in the finder script still uses the Mootools JSON.decode() method.
Seeing Mootools is no longer loaded, the script errors out and the page just hangs.
This patch replaces the JSON.decode() method with the jQuery.parseJSON() method.
When an error occurs during the indexing of pages, this will now handle the error gracefully and return the error message as intended.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
brianteeman
- | Category | ⇒ | JavaScript |
ghazal
- comment
- 5 Feb 2016
Tested OK
wojsmol
- comment
- 5 Feb 2016
@ghazal Please mark your test result on https://issues.joomla.org/tracker/joomla-cms/9078
infograf768
- comment
- 5 Feb 2016
Any easy way to provoke an error when indexing?
nonumber
- comment
- 5 Feb 2016
Have a content plugin throw an error for instance.
| Status | Pending | ⇒ | Ready to Commit |
dgt41
- comment
- 8 Feb 2016
2 good tests -> RTC then
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9078.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-02-10 11:09:07 |
| Closed_By | ⇒ | wilsonge |
| Milestone |
Added: |
||
wilsonge
- comment
- 10 Feb 2016
Thanks Peter :)
I have tested this item
successfully on 6867322
It works OK
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9078.