? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
12 Aug 2017

Pull Request for Issue # .

Summary of Changes

Core.js has some global functions and some really useless one. This PR adds deprecations so we can clean up the file in Joomla 4 (simple proxy).

The only code related change is:

  • Function window.listItemTask is calling Joomla.listItemTask() which is the replacement (simple proxy). This was done in a few more places in this file...

Other than that some CS changes

Testing Instructions

Nothing breaks in the admin

Documentation Changes Required

Core will be fully documented in J4, as the rest of the javascript (hopefully we will get more people in javascript group at some point in the near future)

avatar joomla-cms-bot joomla-cms-bot - change - 12 Aug 2017
Category JavaScript
avatar dgt41 dgt41 - open - 12 Aug 2017
avatar dgt41 dgt41 - change - 12 Aug 2017
Status New Pending
avatar dgt41 dgt41 - change - 12 Aug 2017
Labels Added: ?
2e357e3 12 Aug 2017 avatar dgt41 typo
avatar dgt41
dgt41 - comment - 12 Aug 2017

@mbabker this one is for you :)

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 13 Aug 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Aug 2017

I have tested this item successfully on 2e357e3

Test on List-Views as i suggest "Function window.listItemTask" is for this Views.

If not please explain what to test.


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

avatar dgt41
dgt41 - comment - 13 Aug 2017

@franz-wohlkoenig yes the function is used in the list view, e.g. the publish/unpublish button(or the feature/unfeature) before the title in each row in the articles view.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Aug 2017

Test (un)publish- and (un)feature-Button, works as well.

avatar mbabker mbabker - close - 13 Aug 2017
avatar mbabker mbabker - merge - 13 Aug 2017
avatar mbabker mbabker - change - 13 Aug 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-13 15:58:31
Closed_By mbabker

Add a Comment

Login with GitHub to post a comment