[Xerte-dev] Re: moodle fix

Wed Dec 5 14:26:13 GMT 2012

it's all of those things though.

We don't know who uses XOT and with what, so we can't guess on who uses
stuff. I could make an argument the WordPress plugin would be a world
simpler if Xerte did X and Y, but I wanted to make it work, so I found a
way round it.

If we aren't going to release changes because it breaks X, then what is X.
because we upgraded to PHP 5 and told people to change host if they had a
problem with that

So at present, if we make changes to XOT, we are only going to release them
in they work with Moodle? Which Moodles? For how long?

I don't see why those changes are required for moodle to work, I didn't
remove any of those lines of code, or add them in. I didn't change anything
in modularisation to do with folders (as it doesn't need it). All I changed
in new template was returning one extra parameter in the query - but the
code has broken a long time before that change comes into play.

On Wed, Dec 5, 2012 at 2:03 PM, Ron Mitchell <ronm at mitchellmedia.co.uk>wrote:

> hang on a sec - let's discuss this objectively and practically...****
>
> ** **
>
> We have a very small developer list membership and even fewer of that
> membership testing/developing the latest code on anything but xampp****
>
> ** **
>
> Realistically for organisations to use XOT in earnest they need a public
> facing server install with either ldap or moodle authentication. e.g.
> guest, static or db aren't current/realistic options.****
>
> ** **
>
> I'd wager, but prepared to be proved wrong and perhaps we should do a
> survey, that there are far more xot installations in use that are using
> moodle for authentication than ldap. I certainly know for a fact that there
> are organisations using xot that wouldn't be and wouldn't be able to if
> they weren't able to use moodle for authentication. I know there's no
> internal requirement at Nottingham for Moodle integration but we've had it
> working for years now and it's not a change to moodle code that has stopped
> it working. It's not even a change to the xot moodle integration code that
> has stopped it working. Pat has worked magic in finding a solution to the
> two main issues I've reported but I'm not sure we've really nailed exactly
> what changed to break things in the first place or if it was one thing or
> different things. (see below)****
>
> ** **
>
> These problems with moodle integration didn't exist until last week -
> something has caused a conflict. There's no problem with the previous
> moodle integration in 1.7/1.8 regardless of moodle version.****
>
> ** **
>
> There have been different errors and different issues reported during this
> last week not all of which are down to the same cause/solution. I can only
> reliably report on the issues I've found/tested and it's not always easy or
> efficient to test one thing at a time especially when there are so many
> permutations too. e.g. The blank window and language change issue that Tom
> reported I haven't seen and a fix for Tom re the language setting didn't
> change/fix anything for me. Different problem and  different solution I
> think.****
>
> ** **
>
> Comparing the changes Pat sent through from the current svn code I've
> replaced files individually to test what fixes what:****
>
> ** **
>
> 1. The change to new_template.php fixes the main issue I was seeing e.g.
> not being able to create new projects with a moodle 1.9 integration path
> enabled. I can't see how that change would break non moodle use?****
>
> ** **
>
> 2. The change to your_templates.php fixes the other main issue of the
> workspace not refreshing after creating a project. This was just removal of
> a duplicate require line. Not sure if that was always there, or introduced
> recently, or only a problem with moodle integration in place, but again I
> really can't see how this would break non moodle use?****
>
> ** **
>
> The above two changes fix the main issues I was getting when using an
> install with a moodle 1.9 integration path. The other changes to
> folder_library.php, user_library.php and make_new_folder.php don't seem to
> be needed to fix those main issues as far as I can tell. Not sure if they
> fixes other issues that I haven't seen.****
>
> ** **
>
> Remaining issue****
>
> I am still getting the issue that after creating a new project clicking
> create doesn't work again until after a refresh but that's happening
> without a moodle integration path and as far as I can tell is happening
> with the latest svn code so may be a new problem.****
>
> ** **
>
> HTH****
>
> Ron****
>
> ** **
>
> ** **
>
> ** **
>
> ** **
>
> ** **
>
> *From:* xerte-dev-bounces at lists.nottingham.ac.uk [mailto:
> xerte-dev-bounces at lists.nottingham.ac.uk] *On Behalf Of *Pat Lockley
> *Sent:* 05 December 2012 13:05
>
> *To:* For Xerte technical developers
> *Subject:* [Xerte-dev] Re: moodle fix****
>
> ** **
>
> but the PHP works outside of moodle 1.9 fine
>
> the problem is once with a database and once with a path
>
> Also - we are setting a precedent. We have a working code base, but when
> added into another product, it stops.****
>
> ** **
>
> On Wed, Dec 5, 2012 at 12:47 PM, Dave Burnett <d_b_burnett at hotmail.com>
> wrote:****
>
> The pathing on Moodle 2.0 is a wild and crazy thing.****
>
> Maybe 1.9 was a bit of a test bed for some of what became 2.0?****
>
> Wild guessing.****
>
>  ****
> ------------------------------
>
> Date: Wed, 5 Dec 2012 12:32:09 +0000
> From: patrick.lockley at googlemail.com
> To: xerte-dev at lists.nottingham.ac.uk****
>
>
> Subject: [Xerte-dev] Re: moodle fix
>
> it's not really about introducing errors or changes, but why moodle 1.9
> requires the code to be different? That seems a bit worrying.****
>
> ** **
>
> On Wed, Dec 5, 2012 at 12:27 PM, Ron Mitchell <ronm at mitchellmedia.co.uk>
> wrote:****
>
> I've been testing without a moodle integration path as well as with.****
>
>  ****
>
> This particular window_reference error happens with and without a moodle
> integration path. ****
>
> I'm testing via xampp but you can see it on the techdis install although
> that obviously is using moodle authentication.****
>
>  ****
>
> I haven't found any other errors when testing without integration path so
> I'm not sure your changed files have introduced any problems unless this js
> error is a result of those changes.****
>
>  ****
>
> *From:* xerte-dev-bounces at lists.nottingham.ac.uk [mailto:
> xerte-dev-bounces at lists.nottingham.ac.uk] *On Behalf Of *Pat Lockley
> *Sent:* 05 December 2012 12:14****
>
>
> *To:* For Xerte technical developers
> *Subject:* [Xerte-dev] Re: moodle fix****
>
> ** **
>
>  ****
>
> Yeah, my bad - not sure why that would happen. Nothing I've changed
> javascript side.
>
> I wouldn't commit those changes as they just don't make sense to me. I
> don't get why moodle inclusion would cause those problems and I am not sure
> what they do outside of moodle.****
>
>  ****
>
> On Wed, Dec 5, 2012 at 12:13 PM, Ron Mitchell <ronm at mitchellmedia.co.uk>
> wrote:****
>
> I thought that window_reference is undefined is a js error?****
>
>  ****
>
> BTW****
>
> Are you going to commit those changed php files or do you want me to on
> your behalf?****
>
>  ****
>
> *From:* xerte-dev-bounces at lists.nottingham.ac.uk [mailto:
> xerte-dev-bounces at lists.nottingham.ac.uk] *On Behalf Of *Pat Lockley
> *Sent:* 05 December 2012 12:02****
>
>
> *To:* For Xerte technical developers
> *Subject:* [Xerte-dev] Re: moodle fix****
>
>  ****
>
> Not sure then, I assumed it was that (as this was more of a patch than a
> fix)
>
> Do you get a JS error?****
>
>  ****
>
> On Wed, Dec 5, 2012 at 11:52 AM, Ron Mitchell <ronm at mitchellmedia.co.uk>
> wrote:****
>
> Just print_r($_SESSION) or could there be a similar print_r(something)
> causing the same problem?****
>
> Have searched for print_r($_SESSION) in the install I'm using for testing
> and not found.****
>
>  ****
>
> *From:* xerte-dev-bounces at lists.nottingham.ac.uk [mailto:
> xerte-dev-bounces at lists.nottingham.ac.uk] *On Behalf Of *Pat Lockley
> *Sent:* 05 December 2012 11:42****
>
>
> *To:* For Xerte technical developers
> *Subject:* [Xerte-dev] Re: moodle fix****
>
>  ****
>
> I think it's because of the print_r($_SESSION) somewhere - remove that and
> it'll be fine****
>
>  ****
>
> On Wed, Dec 5, 2012 at 11:36 AM, Ron Mitchell <ronm at mitchellmedia.co.uk>
> wrote:****
>
> Hi Pat****
>
> thanks for the update.****
>
> I've been testing on both the techdis server and locally via xampp.
> Obviously I won't have tested every single permutation yet but from tests
> so far all seems fine apart from this issue of not being able to create a
> new project after creating one already until refresh. However this happens
> without a moodle integration path added too.****
>
>  ****
>
> Here's the steps to reproduce with current svn code plus your changed
> files.****
>
> Click create to create a new project based on Xerte Online Toolkits
> template****
>
> Close the editor and try to create another project - nothing happens upon
> clicking create****
>
> Refresh the browser and click create works but the cycle repeats e.g.
> can't create a second project****
>
>  ****
>
> Using debugger in IE9 I get the following error when closing the editor
> window:****
>
>  ****
>
> SCRIPT5009: 'window_reference' is undefined ****
>
> edit.php?template_id=17, line 32 character 1****
>
>  ****
>
> Not really sure where to look for the cause/solution to this one but is
> there perhaps an include/require missing somewhere?****
>
>  ****
>
> Cheers****
>
> Ron****
>
>  ****
>
>  ****
>
> *From:* xerte-dev-bounces at lists.nottingham.ac.uk [mailto:
> xerte-dev-bounces at lists.nottingham.ac.uk] *On Behalf Of *Pat Lockley
> *Sent:* 05 December 2012 09:38****
>
>
> *To:* For Xerte technical developers****
>
> *Subject:* [Xerte-dev] Re: moodle fix****
>
>  ****
>
> the problem in new template was that for some reason moodle is leaving or
> creating another database link so when toolkits sends the request to the
> database it fails as the code is now pointing at the wrong database.
>
> Added in another database_connect call to fix this
>
> the problem in new folder creation was two require_once("config.php")
> being used. Removed one and it works.
>
> Why these would only affect moodle 1.9? No idea.****
>
>  ****
>
> On Wed, Dec 5, 2012 at 7:49 AM, Ron Mitchell <ronm at mitchellmedia.co.uk>
> wrote:****
>
> Hi Pat****
>
> thanks - from tests so far those files have certainly improved things!****
>
>  ****
>
> What did you find/change? What was the main cause of the problem?****
>
>  ****
>
> I've tested creating new projects and folders, moving and deleting
> projects all of which were a problem previously so looking hopeful.****
>
>  ****
>
> One problem/difference at the moment which I don't think was there before
> is that with a moodle 1.9 integration path I can now create a new project
> without the workspace refreshing and it appears etc but if I then try to
> create a second project clicking create to create another Xerte Online
> Toolkit project doesn't do anything. I have to refresh the browser before I
> can create another project. Clicking to create an RSS project does work
> without refresh and with moodle 2 configured I can create a second and
> third template etc without problem.****
>
>  ****
>
> I've only tested this locally so far will test on the techdis server
> next...****
>
>  ****
>
> Cheers****
>
> Ron****
>
>  ****
>
> *From:* xerte-dev-bounces at lists.nottingham.ac.uk [mailto:
> xerte-dev-bounces at lists.nottingham.ac.uk] *On Behalf Of *Pat Lockley
> *Sent:* 05 December 2012 00:29
> *To:* For Xerte technical developers
> *Subject:* [Xerte-dev] moodle fix****
>
>  ****
>
> try swapping these files ****
>
>  ****
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
>  ****
>
>  ****
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
>  ****
>
>  ****
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
>  ****
>
> ** **
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
>  ****
>
>
>
> ****
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
> ** **
>
>
> _______________________________________________ Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
>
>
> ****
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev****
>
> ** **
>
>
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.nottingham.ac.uk/pipermail/xerte-dev/attachments/20121205/d6fb4efe/attachment-0001.html>