[geeklog-devel] Add a new feature handle the Ajax request

[Tom]

I like Kenji idea about adding an ajax function to the scripts class. This
way other plugins etc... will be able to use it.

We might as well add in the Filemanager and your new feature/bug fix to
Geeklog 2.1.0. I haven't finished up my bug list yet (I've gotten side track
by work) and so it is probably another week or 2 until our first release.

If anyone has a chance can they take a look at 

http://project.geeklog.net/tracking/view.php?id=1715

Thanks

Tom




-----Original Message-----
From: geeklog-devel-bounces at lists.geeklog.net
[mailto:geeklog-devel-bounces at lists.geeklog.net] On Behalf Of Kenji ITO
Sent: January-04-14 6:12 PM
To: Geeklog Development
Subject: Re: [geeklog-devel] Add a new feature handle the Ajax request

Yoshinori wrote:

> I want to add a new feature that handle the Ajax request into Geeklog.
> I think Ajax is effective to solve a bug #0001543.
>
> http://project.geeklog.net/tracking/view.php?id=1543

I don't fully agree.  The same can be done with PHP.

> In order to achieve this, I want to minor modifications of 
> lib-common.php and add new files, /javascript/ajax_parts.js 
> /system/lib-ajax.php

Since JavaScript is controlled by the scripts class in "scripts.class.php",
why don't you add a new method to the class? For
example:

public function ajax($url, $method = 'GET', $parameters = array())

> This fix is so small, so I believe there is no problem even if 
> committed to the HEAD of the repository.
> If there is any problem, then I will revert immediately.
> Or, should I create a new branch?

I believe there will be no problem, too.
--
Kenji ITO
http://mystral-kk.net/


_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://eight.pairlist.net/mailman/listinfo/geeklog-devel