2.0 Preview 3 - glossary autolink problem

2.0 Preview 3 - glossary autolink problem

by Itamar Tzadok -
Number of replies: 4
When clicking an autolinked glossary term (in a label or a page resource) I get 'Error reading from database'. Has anyone experienced something similar? smile
Average of ratings: -
In reply to Itamar Tzadok

Re: 2.0 Preview 3 - glossary autolink problem

by Joseph Rézeau -
Picture of Core developers Picture of Particularly helpful Moodlers Picture of Plugin developers Picture of Testers Picture of Translators
I am getting a similar error: "error/dmlreadexception".
Joseph
In reply to Itamar Tzadok

Re: 2.0 Preview 3 - glossary autolink problem

by Eloy Lafuente (stronk7) -
Picture of Core developers Picture of Documentation writers Picture of Moodle HQ Picture of Peer reviewers Picture of Plugin developers Picture of Testers
Hi,

I've committed a fix for that problem (there was one small typo in one query). Should be working now.

Ciao smile
In reply to Eloy Lafuente (stronk7)

Re: 2.0 Preview 3 - glossary autolink problem

by Mauno Korpelainen -

mixedThanks Eloy,

it fixed the glossary autolink popup issue - my test site still gets a notice:

Notice: Undefined property: stdClass::$rating in C:\xampp\xampp\htdocs\moodle\moodle\mod\glossary\lib.php on line 1097

and if I change in function  glossary_print_entry_lower_section line 1097 from

    if ($aliases || $icons || $entry->rating) {

to

    if ($aliases || $icons || !empty($entry->rating)) {

at least this notice is gone...

I did not find any bug report for this issue so I added this note here.

EDIT: Wait a minute - the glossary item is not at all shown in a popup window but has still Close this window button mixed

In reply to Mauno Korpelainen

Re: 2.0 Preview 3 - glossary autolink problem

by Eloy Lafuente (stronk7) -
Picture of Core developers Picture of Documentation writers Picture of Moodle HQ Picture of Peer reviewers Picture of Plugin developers Picture of Testers
Yes, yes Mauno, I also detected exactly those (the notice and the close things).

Was creating the bug in the Tracker right now: MDL-22678

Ciao smile