[Fixed - maybe] Weird course display bug

More
14 years 5 months ago #1 by Jason
Hey guys/gals.

So the installation of joomdle I've been using is somewhat custom - forked since, oh 1.4. But I found an interesting bug in the code that may or may not be in 1.6.

When looking at the enrolment page for a course, if no start and end date for the course was listed - the enrolment would often look like the following:

(example)
Start Date: 22-09-2011
Enrolment start date: 31-12-1969
Enrolment end date: 31-12-1969
Enrolment duration: Unlimited

Showing 1969. This is due the php code in the detail view using the php function arraykeyexists. Noting that this function doesnt work with objects since php 5.3 ( php.net/manual/en/function.array-key-exists.php ) once I replaced it with the function property_exists, the error went away.

So, just thought i'd let ya'll know.

Thanks,
Jason.

Please Log in or Create an account to join the conversation.

More
14 years 5 months ago #2 by Chris
Thanks very much for feeding this back.

Out of interest what where the features you needed to fork in Joomdle. Perhaps these could be included in the core so that you would not need to worry about using a forked version and so that others could benefit too.

Regards,

Chris

Please Log in or Create an account to join the conversation.

More
14 years 5 months ago #3 by Jason
I modified the views to add some elements that made styling easier, display of large course categories better, and changed a little how students enrolled in courses. But really, at the time we didn't have a great change control system on our web-servers and after I customized 0.4 and the new versions came out, I didn't have the time to merge them manually. BUT, now we have git, and I've decided today's project is going to be merging to the latest version.

Please Log in or Create an account to join the conversation.

More
14 years 5 months ago #4 by Antonio Durán
Replied by Antonio Durán on topic Re: [Fixed - maybe] Weird course display bug
Thans for the tip, I will review the code to see if the bug still exists.

Also, it would be great if you could share your changes on styling, as we are not really into design, and could definetly use some help there :)

Antonio

Please Log in or Create an account to join the conversation.