?
avatar kirapwn
kirapwn
28 Oct 2014

Steps to reproduce the issue

Try to create module AJAX request for administrator app

Expected result

Run AJAX on admin module

Actual result

Looks only for site modules, using include in admin

Additional comments

The class should be refactored to add admin AJAX module support and to use Joomla! loader

avatar kirapwn kirapwn - open - 28 Oct 2014
avatar Bakual
Bakual - comment - 28 Oct 2014

@betweenbrain Something for your to look into.

avatar kirapwn
kirapwn - comment - 28 Oct 2014

I can create PR, but I would need to talk about refining component on site part. As it is all in one file right now and I am not sure which way would be the best:

-load helper class and fire it if no variable is passed for compatibility
-go MVC which may be too much just for module/plugin file load and call
-leave it as it is and just fix backend to call proper path files

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

avatar betweenbrain
betweenbrain - comment - 29 Oct 2014

@kirapwn Can you explain more about what you mean by

load helper class and fire it if no variable is passed for compatibility

avatar kirapwn
kirapwn - comment - 29 Oct 2014

What I mean by this is to put all the logic from current ajax.php file to helper class and create new instance of it inside main file to do what it does right now. Also to add a variable for admin/site choose which defaults to site. This way we can choose if we want to run administrator module AJAX or if we want default behavior.

avatar brianteeman brianteeman - change - 3 Jan 2015
Labels Added: ?
avatar brianteeman brianteeman - change - 12 Nov 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-11-12 17:43:57
Closed_By brianteeman
avatar brianteeman brianteeman - close - 12 Nov 2015
avatar brianteeman brianteeman - close - 12 Nov 2015
avatar brianteeman
brianteeman - comment - 12 Nov 2015

It has been over a year since the last comment on this and with no code to evaluate I am closing this. It can always be reopened if in the future someone wants to propose some code


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

Add a Comment

Login with GitHub to post a comment