Issue Details (XML | Word | Printable)

Key: MDL-10538
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Not a bug
Priority: Major Major
Assignee: Petr Skoda
Reporter: Mitsuhiro Yoshida
Votes: 4
Watchers: 9
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

IE can't open powerpoint resource file type.

Created: 20/Jul/07 06:59 AM   Updated: 27/Feb/08 05:04 PM
Return to search
Component/s: Resource
Affects Version/s: 1.9
Fix Version/s: None

File Attachments: 1. Text File 0001-filelib-Work-around-IE-Powerpoint2007-download-bug.patch (2 kB)
2. Text File 0001-filelib-Work-around-IE-Powerpoint2007-download-bug.patch (2 kB)
3. Text File resource.class.php.patch (0.7 kB)
4. File revision53.diff (3 kB)

Issue Links:
Duplicate
 
Relates
 

Participants: A. T. Wyatt, Daniel Pouliot, Kristian Thornley, Martín Langhoff, Mitsuhiro Yoshida, Nicolas Connault, Petr Skoda and Rob Faulkner
Security Level: None
Resolved date: 05/Feb/08
Affected Branches: MOODLE_19_STABLE


 Description  « Hide
When we click on the following powerpoint file resouce, a blank page will be displayed.
 1. Resource type: ppt (Powerpoint file)
 2. Window: new window

To prevent this blank page, we can modify mod/resource/type/file/resource.class.php as below.

Index: resource.class.php
===================================================================
RCS file: /cvsroot/moodle/moodle/mod/resource/type/file/resource.class.php,v
retrieving revision 1.68
diff -u -r1.68 resource.class.php
--- resource.class.php 19 Jul 2007 05:51:33 -0000 1.68
+++ resource.class.php 19 Jul 2007 22:51:38 -0000
@@ -605,6 +605,9 @@
             echo '</div>';
             echo "<div class=\"popupnotice\">(<a href=\"$fullurl\">$fullurl</a>)</div>";
         }
+ if ($mimetype == "application/vnd.ms-powerpoint") {
+ $fullurl = $fullurl . "?forcedownload=1";
+ }
         redirect($fullurl);
     }



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
No commits have yet been performed on this issue.