PostDownload Hook benutzen

[feG3]

class DownloadResponseListener
{
    public function __construct(
        private readonly string $projectDir,
        private readonly string $uploadPath,
    ) {
        
    }
} 

# config/services.yaml
services:
    _defaults:
        autowire: true
        autoconfigure: true
        bind:
            $projectDir: '%kernel.project_dir%'
            $uploadPath: '%contao.upload_path%'

    App\:
        resource: ../src 

Irgendwie bekomm ich das so nciht zum Laufen, bekomme immer den Fehler:

Cannot autowire service "App\EventListener\DownloadResponseListener": argument "$projectDir" of method "__construct()" is type-hinted "string", you should configure its value explicitly.

muss ich die services.yaml noch irgendwo spezifisch einbinden oder so?

[Spooky]

Poste deinen PHP Code und deine services.yaml

[feG3]

Poste deinen PHP Code und deine services.yaml

Ich war einfach dämlich....

Ich hab die services.yaml in 'contao/config/' gepackt statt in 'config'....

Ist mir gerade aufgefallen und nach dem Verschieben geht's.. Es wird Zeit für's Wochenende

[BugBuster]

Hab den Hook bei mir auch umgebaut als DownloadResponseListener.
Funktioniert super.

Bis auch eine witzige Kleinigkeit.
Wenn der Browser das in der Startseite defininierte Favicon anfordert, wird das auch als BinaryFileResponse durchgeführt, wodurch ich diesen "Download" nun zähle.
Das will ich natürlich nicht, und ich Frage mich grad was da noch alles nebenbei gezählt werden könnte.

Kann ich das verhindern, ohne jetzt explizit favicon.ico und das was in der Startseite als solches eingetragen, ist zu filtern?

[BugBuster]

Hab das mal grob protokolliert, eventuell könnte ich nach der Seiten-ID=0 filtern.
Bildschirmfoto vom 2023-12-11 02-06-32.png

[BugBuster]

Jo, funktioniert.

if (!$pageModel instanceof PageModel) {
  return;
}

[Spooky]

Was genau meinst du damit? Ich denke wenn dann musst du den Datei Pfad prüfen.

[BugBuster]

Wenn der Browser ein "GET /favicon.ico" anfordert, ist das auch ein "BinaryFileResponse".
Den Zugriff auf die ico will ich aber gar nicht zählen, ist ja kein Download über ein Download(s) Element.

Nach Datei Name kann ich nicht gehen, den bekomme ich zwar, aber ob das nun per Klick oder über andere Wege kam, sehe ich ja darüber nicht.
Bei einen echten Download bekomme ich eine Page ID (also ein PageModel) und somit ist es ein echter Download.

Daher:

$pageModel = $event->getRequest()->attributes->get('pageModel');
if (!$pageModel instanceof PageModel) {
 return;
}

Das funktioniert bei mir, zumindest taucht nun das favicon.ico nicht mehr auf in der Download Zählung.

Weiter oben meintest du ja, man muss das filtern:

Darüberhinaus ein early out einbauen für den Fall, dass der Dateipfad nicht mit dem Parameter contao.upload_path beginnt ("files").

Das wäre die Alternative, über Pfad zu gehen und mit contao.upload_path zu vergleichen.

Ich habs nun über das PageModel gemacht, habe ich mir abgeschaut, eigentlich wollte ich prüfen ob PageModel->id = 0 ist, ist es in so einem Fall, aber das war dann schon ein Schritt zu weit.

[Spooky]

Bei einen echten Download bekomme ich eine Page ID (also ein PageModel) und somit ist es ein echter Download.

Nur über das Download(s) Inhaltselement. Und auch nur über das alte, oder?

[BugBuster]

Wenn Contao 5.2 die neuen mitbringt, dann bei den neuen Inhaltselementen.