search

chamilo / chamilo-lms

Chamilo is a learning management system focused on ease of use and accessibility
https://chamilo.org
GNU General Public License v3.0
811 stars 481 forks source link

PDF reader does not display document in 1.11.28 #5917

Closed oneill78 closed 4 days ago

oneill78 commented 2 weeks ago

Hello, The PDF reader does not display the document in 1.11.28, while it worked correctly in 1.11.26. There is a "download" button that appears at the top of the page in the learning path but not the PDF reader.

For information, the PDF is directly added in the left menu in the path.

How can I fix this?

This is important because the learning paths are mostly made up of PDFs

Thank you for your answer

Bavarianspirit commented 1 week ago

We have the same issue.

AyaMaksoud commented 1 week ago

Same as #5907

We have the same issue. Rendering pdf is necessary. So in order to unlock the situation untel the correction is made, I removed the pdf from being downloaded, so it is always displayed.

In main\lp\learnpath.class.php, I removed "pdf" from the list $extensionsToDownload.

NicoDucou commented 1 week ago

Hi all The correction from @AyaMaksoud is correct. We just modified yesterday in the following commit : https://github.com/chamilo/chamilo-lms/commit/54d55b59b46e727aeafbe2243bcf85db09d09289

So apply this commit and it should work as before.

Best regards

Bavarianspirit commented 1 week ago

works for me. Thanks

oneill78 commented 4 days ago

Hello, Thanks for the fix. Everything works fine. Lionel