PDF im Browser anzeigen

[umsteiger]

Hallo,
nachdem das mit dem Download-Verzeichnis geklärt ist, kommt mir gleich die nächste Frage: Wie kann ich Contao dazu veranlassen, die pdf-Datei direkt im Browser (bei entsprechend aktiviertem Browser-Plugin) anzuzeigen, ohne dass der Download gestartet wird?

umsteiger

[lucina]

Aus dem Bauch heraus:

<FilesMatch "\.(?i:pdf)$">
    ForceType application/octet-stream
    Header set Content-Disposition inline
</FilesMatch>

[umsteiger]

Schön, schön, und wohin damit?

[lucina]

Wo werden denn Header gesetzt?

;-)

[umsteiger]

Hallo,
das mit der Ergänzung in htaccess hat leider nicht geklappt. Ich habe das Problem etwas eingegrenzt:
Im Template ce_downloads steht:

<a href="<?= $file['href'] ?>

, welches in meinem Fall jetzt als Link dieses hier produziert:

<a href="index.php/magazin.html?file=files/magazin/magazin_2016_01.pdf"

. Und das ruft den Download-Dialog auf. Jetzt habe ich

<?= $file['href'] ?

durch folgendes ersetzt:

files/magazin/magazin_2016_01.pdf

und es öffnet sich die pdf-Datei im Browser, ohne htaccess-Anpassung.
Jetzt stellt sich die Frage, wie bekomme ich nur den Pfad zur Datei angezeigt ohne

index.php/magazin.html?file=

.

Gruß
umsteiger

[Andreas]

Beispiel:

<?php foreach ($this->files as $file): ?>
<?php
  /* Neuer Code */
  #dump($file['href']);
  if($file['extension'] == 'pdf')
  {
    $file['href'] = str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name'];
  }
  #dump($file['href']);
?>

Das index.php bekommst du weg, indem du die .htaccess.default umbenennst nach .htaccess und in den Einstellungen die URL-Umschreibung aktivierst.

[umsteiger]

Danke, das war die Lösung.
Ich habe jetzt einfach in ce_downloads das

$file['href']

durch

$file['href'] = str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name']

ersetzt und alles funzt.

umsteiger

[Andreas]

Das wäre irgendwie nicht ganz richtig. Wenn du es für alle Dateitypen möchtest, dann entweder so

<?php foreach ($this->files as $file): ?>
<?php
  $file['href'] = str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name'];
?>

Oder so

...<a href="<?= $file['href'] ?>" title=...

Neu:
...<a href="<?= str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name'] ?>" title=...

[umsteiger]

OK, da es nur um pdf geht, habe ich die zweite Variante genommen.

Danke schön.
umsteiger

[tomkarp]

Ich hatte genau das gleiche Problem und es so gemacht:

Danke, das war die Lösung.
Ich habe jetzt einfach in ce_downloads das

$file['href']

durch

$file['href'] = str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name']

ersetzt und alles funzt.

Funktioniert auch anscheinend alles wie es soll.

Inwiefern ist die erste Lösung nicht richtig, wie hier geschrieben:

Das wäre irgendwie nicht ganz richtig. Wenn du es für alle Dateitypen möchtest, dann entweder so

<?php foreach ($this->files as $file): ?>
<?php
  $file['href'] = str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name'];
?>

Oder so

...<a href="<?= $file['href'] ?>" title=...

Neu:
...<a href="<?= str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name'] ?>" title=...

Viele Grüße, Tom

[Andreas]

Das was in Post #8 steht ist beides richtig, das obere finde ich persönlich übersichtlicher, weil ich die Variable $file['href'] dort separat neu befülle und an dieser Stelle auch nochmal eine Einschränkung auf bestimmte Dateitypen machen könnte. An dieser Stelle könnte ich auch noch einen Kommentar einfügen um später schnell zu sehen, was geändert wurde. Und weil die Stelle an welcher die Variable $file['href'] eingebunden wird so belassen wird wie sie im O-Template ist.

Was nicht ganz richtig gewesen wäre war das, was @umsteiger in #7 geschrieben hat. Danach würde es in seinem Template so aussehen:

Alt:
...<a href="<?= $file['href'] ?>" title=...

Neu:
...<a href="<?= $file['href'] = str_replace(TL_ROOT.'/', '', $file['path']).'/'.$file['name'] ?>" title=...

[Andreas]

Ach so, schon wieder vergessen zu sagen. Dadurch verliert man den Schutz der Dateien auf geschützten Seiten. Wer den Link kennt, kommt an die Datei ran. Das dürfte eigentlich mit dem Original-Template nicht der Fall sein.

[x-ray]

Ich habe es bei mir mit folgender Änderung in der .../library/Contao/File.php hinbekommen (auch wenn ich noch nicht weiß, ob es Seiteneffekte hat).

Vorher:

    header('Content-Disposition: attachment; filename="' . ($filename ?: $this->basename) . '"'); 

Nachher:

if ($this->extension == 'pdf'){
    header('Content-Disposition: inline; filename="' . ($filename ?: $this->basename) . '"');
} else {
    header('Content-Disposition: attachment; filename="' . ($filename ?: $this->basename) . '"');
} 

[Thraile]

Nunja, update-sicher ist das jedenfalls nicht

Gesendet von meinem D5803 mit Tapatalk

[x-ray]

Stimmt, update sicher ist es nicht. Ich glaube aber in Version 4 ist das aber besser umgesetzt. Zumindest war auf der Contao Demo das so, dass sich PDFs im Browser öffnen


Gesendet von iPhone mit Tapatalk

[tab]

Echt? Auf welcher Seite der Demo ist denn das? Die einzigen Downloads, die ich auf die Schnelle da finde, sind Bilder (.jpg). Und nicht mal die werden im Browser geöffnet.

[Andreas]

Und wenn ich meinen Browser jetzt so einstelle, dass er mir genau diese Anwort als Download anbieten soll? Dann hast du eine Anpassung im Core, die nichts bringt. Die Templateanpassung ist die einfache und updatesichere Lösung.

Was meinst du mit in C4 ist es besser? Wird die Datei dort auch per PHP ausgeliefert um so das Ausliefern aus gesperrten Verzeichnissen anzubieten aber es wird ein anderer Header gesendet, der die default Browsereinstellung nicht dazu animiert einen Download anzubieten?

[x-ray]

Echt? Auf welcher Seite der Demo ist denn das? Die einzigen Downloads, die ich auf die Schnelle da finde, sind Bilder (.jpg). Und nicht mal die werden im Browser geöffnet.

Ich hatte eine Seite im Editor bearbeitet und einen Link eimgebaut der auf eine PDF aus dem Dateien von der Demo zeigt.

Im Frontend hat ein Klick auf den Link die Datei inline im Brower geladen und angezeigt.

Im gleichen Browser ohne irgendwelche Einstellungen des Browseres zu ändern, wurde auf meiner Contao 3.5 ein klick auf solch einem Link mit dem Download der Datei beantwortet.


Gesendet von iPhone mit Tapatalk

[x-ray]

Und wenn ich meinen Browser jetzt so einstelle, dass er mir genau diese Anwort als Download anbieten soll? Dann hast du eine Anpassung im Core, die nichts bringt. Die Templateanpassung ist die einfache und updatesichere Lösung.

Was meinst du mit in C4 ist es besser? Wird die Datei dort auch per PHP ausgeliefert um so das Ausliefern aus gesperrten Verzeichnissen anzubieten aber es wird ein anderer Header gesendet, der die default Browsereinstellung nicht dazu animiert einen Download anzubieten?

Ich weiß nicht was man am Browser konfigurieren muss, dass er PDFs generell herunterlädt.

Der Kunde wünschte, dass ein Klick auf den PDF Link inline im Browser die PDF anzeigt. Wie das in Contao 3.5 und 4 genau funktioniert kann ich nicht sagen. Vermute beides wird mit PHP ausgeliefert und es wird beim Senden eine Header Eigenschaft gesetzt auf die der Client (Browser) reagiert.

Das ist erst mal der Default der vom Server kommt und ja man kann vermutlich auch im
browser was erzwingen. Ähnlich wie "links in neuem Fenster öffnen" erwingt ja auch ein neues Tab oder fenster obwohl kein target blank im a-Tag definiert wurde


Gesendet von iPhone mit Tapatalk

[tab]

Na ein Link auf ein PDF wird mir auch im Browser angezeigt - sofern der Browser das kann, was alles andere als selbstverständlich ist. Ein Download dagegen lässt mich speichern oder in einem Programm meiner Wahl anzeigen.

[Andreas]

Ich weiß nicht was man am Browser konfigurieren muss, dass er PDFs generell herunterlädt.

Screen-2017-01-24_23-23-04.png