[Xerte-dev] Re: moodle fix

Wed Dec 5 14:03:57 GMT 2012

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.nottingham.ac.uk/pipermail/xerte-dev/attachments/20121205/4ed0d4b4/attachment-0001.html>