andreknieriem / photobooth

A photobooth Web-Application for raspberry pi with gphoto2
https://photobooth.andrerinas.de/
MIT License
300 stars 163 forks source link

Feature request: Make QR Code optional on print #66

Closed ChrisKram closed 5 years ago

ChrisKram commented 5 years ago

Im Moment kann ein Bild per Default nur mit QR Code gedruckt werden. Dafür wird das Bild kleiner dargestellt und der QR Code rechts neben das Foto gesetzt.

Die ist im Moment, da der Barcode keinen externen Link in die Cloud enthält nur während der Veranstaltung verwendbar und somit eher schade um das Bild. Daher wäre es praktisch, den Barcode optional zu gestalten und per Admin Panel für den Print ein oder ausschaltbar zu machen.

In english please: Actually the qr codes is printed onto the foto by default. A resizing happens and the qr code is added to the left.

Since the barcode is only useful while the party happens it would be great when the barcode is optional and an additional option in the admin panel allows it to deactivate it.

andi34 commented 5 years ago

Wie bereits im anderen Issue schon erwähnt (keine Rückmeldung erhalten): Versuch Mal diiese Line auszukommentieren: https://github.com/andreknieriem/photobooth/blob/master/print.php#L51

Also

// imagecopyresized($print, $code, $width, 0, 0, 0, ($height / 2), ($height / 2), imagesx($code), imagesy($code));

Könnte ausreichen auf die schnelle betrachtet.

Ist kein Thema das dann optional zu machen wenn's reicht, bekomme ich auch kurzfristig am Wochenende ins Admin Menü gepackt.

ChrisKram commented 5 years ago

Ich weiß noch nicht ob ich es heute oder morgen noch schaffe, brauche die Box am WE und muss noch andere Dinge ändern, bevor es losgeht.

flighter18 commented 5 years ago

Hi @ChrisKram

Ich wollte auch, dass nur das Bild ohne QR Code gedruckt wird. Um dies zu erreichen, musste ich diverse Zeilen in der print.php Datei auskommentieren. Zusätzlich, musste ich eine Zeile hinzufügen.

Im Anhang meine print.php Datei. Vielleicht hilft Dir das weiter. Bitte beachte, ich musste die Dateiendung von php auf txt ändern.

print.txt

ChrisKram commented 5 years ago
// imagecopyresized($print, $code, $width, 0, 0, 0, ($height / 2), ($height / 2), imagesx($code), imagesy($code));

Könnte ausreichen auf die schnelle betrachtet.

@andi34 das reicht leider nicht aus. Dann ist zwar der QR Code nicht mit auf dem Bild, es wird aber weiterhin verkleinert. Ich denke, dass es dafür eine else Bedingung braucht in der dann nur der print job aufgerufen wird.

Ungefähr so (habs nicht ausprobiert und bin nicht vertraut mit php - bitte noch mal prüfen) - siehe Anhang. print.txt

@flighter18 - danke für den Vorschlag. ich habe mir eine ähnliche Version zusammen gebaut. Siehe #64

andi34 commented 5 years ago

Danke @ChrisKram und @flighter18 , ich schau Sonntag nochmal rein

andi34 commented 5 years ago

@ChrisKram falls du testen willst

cd /var/www/html
git config core.fileMode false
git add --all && git commit -a -m "Squash"
git remote add andi34 https://github.com/andi34/photobooth
git fetch andi34 dev
git checkout andi34/dev
sudo mkdir -p /var/www/html/images
sudo mkdir -p /var/www/html/keying
sudo mkdir -p /var/www/html/print
sudo mkdir -p /var/www/html/qrcodes
sudo mkdir -p /var/www/html/thumbs
sudo mkdir -p /var/www/html/tmp
sudo chown -R pi: /var/www/
sudo chmod -R 777 /var/www
andi34 commented 5 years ago

Works fine. Merged.