Hia, I noticed that the word "Contents" was always showing at the bottom of all my nodes which had a SCORM field, even when there was no SCO uploaded.

I "fixed" this by moving line 354 in player.php to within the if-statement below (see attached screenshot). However, I haven't really found out when this line *should* be printed so it might be the wrong approach...(?)

CommentFileSizeAuthor
#3 cckcontents2.png388.86 KBvegardjo
#2 cck_scormcontents.png433.79 KBvegardjo
scorm-contents.jpg61.38 KBvegardjo

Comments

pgerling’s picture

pgerling commented
Assigned: Unassigned » pgerling
Status: Active » Closed (cannot reproduce)

i cannot reproduce this issue - the word "contents" never appears for me (even with a clean drupal installation)

vegardjo’s picture

vegardjo commented
Version: 6.x-1.0-beta6 » 6.x-1.0-beta8
Status: Closed (cannot reproduce) » Active
StatusFileSize
new cck_scormcontents.png433.79 KB

Hm, you don't? I just updated to the latest beta, and the bug is back, you can see it live here: http://www.helsekompetanse.no/mangfold/736 (I just picked a page, but you will see it in all pages there, and none of them has any SCORM content attached. But they are all book pages, and that content type has the CCK Scorm field set up.

See also screenshot in case I fix this again on the live site before you get a chance to see it..

Here are lines 356-7 in player.php where it is set:

    //print_simple_box_start('center',$boxwidth);
    $output='<div class="structurehead">Contents</div>';
vegardjo’s picture

vegardjo commented
StatusFileSize
new cckcontents2.png388.86 KB

I tried on a fresh install too, and couldn't reproduce it initially, but when I added my book page to a menu (primary links in this case) the bug is back, along with this warning:

warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /Applications/MAMP/htdocs/clean/sites/all/modules/cck_scorm/player.php on line 362.

See screenshot.

pgerling’s picture

pgerling commented
Status: Active » Needs review

Now that's strange. I still cannot reproduce your issue, but since I can clearly see it on your website and screenshots - and because I have no idea why "Contents" should appear there anyways - I excluded line 357 from player.php. You can find this fix in the new beta9. Thanks a lot for your help! :o)

vegardjo’s picture

vegardjo commented
Version: 6.x-1.0-beta8 » 6.x-1.0-beta9
Status: Needs review » Reviewed & tested by the community

Thanks - I have updated to beta9 and "Contents" no longer shows :)

..however the php warning is still there

pgerling’s picture

pgerling commented
Version: 6.x-1.0-beta9 » 6.x-1.0-beta10
Status: Reviewed & tested by the community » Closed (cannot reproduce)

We could not reproduce this bug in beta10