Project

General

Profile

Bug #7271

Images gallery in documents is disabled

Added by Yannick Warnier about 5 years ago. Updated about 5 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Documents
Target version:
Start date:
19/09/2014
Due date:
% Done:

100%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
2

Description

In the documents tool, there was an "image gallery" feature available in versions up to Chamilo 1.9.6, but now it doesn't appear anymore.

The icon is main/img/icons/32/slideshow.png, and it should appear in the action bar of the documents menu as soon as there are images (png or jpg) available in the current folder.

See line 1656 of main/document/document.php:

// Slideshow by Patrick Cool, May 2004
require 'document_slideshow.inc.php';
if ($image_present && !isset($_GET['keyword'])) {
    $actions .= Display::url(
        Display::return_icon('slideshow.png', get_lang('ViewSlideshow'), '', ICON_SIZE_MEDIUM),
        api_get_path(WEB_CODE_PATH).'document/slideshow.php?'.api_get_cidreq().'&curdirpath='.$curdirpathurl
    );
}

Somehow, it seems that $image_present is never true anymore.

Associated revisions

Revision 7a36a946 (diff)
Added by Angel Fernando Quiroz Campos about 5 years ago

Enable image gallery in the documents tool - refs #7271

Revision 04e77165
Added by Yannick Warnier about 5 years ago

Merge pull request #365 from AngelFQC/7271

Enable image gallery in the documents tool - refs #7271

History

#1

Updated by Angel Quiroz about 5 years ago

  • Status changed from Assigned to Needs testing
  • % Done changed from 0 to 50

Enable the image gallery in the documents tool
I fix this, renaming the $document_and_folders variable to $documentAndFolders in the main/document/document_slideshow.inc.php file for search if there are images in the folder that was set in the main/document/document.php file

PR: https://github.com/chamilo/chamilo-lms/pull/365

#2

Updated by Angel Quiroz about 5 years ago

  • Assignee deleted (Angel Quiroz)
#3

Updated by Yannick Warnier about 5 years ago

  • SCRUM pts - complexity changed from ? to 2
#4

Updated by Yannick Warnier about 5 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee set to Angel Quiroz
  • % Done changed from 50 to 100

Reviewed, approved and closing

Also available in: Atom PDF