ariselseng / camerarawpreviews

Camera Raw Previews app for Nextcloud
GNU Affero General Public License v3.0
52 stars 11 forks source link

0.8.5 update seems to break preview on Nextcloud 28 #105

Closed helloha82 closed 3 months ago

helloha82 commented 3 months ago

Just updated to 0.8.5 with an update to NC29 in mind but decided to hold off until the next NC dot release.

Now 0.8.5 does not seem to work with NC28 anymore?

ariselseng commented 3 months ago

Which php are you running?

plaes commented 3 months ago

I'm also seeing some issue with nextcloud 29.0.1 + camerarawpreviews-0.8.5, firstly there's a JS error in console:

The handler doesn't have a valid component 
Object { id: "camerarawpreviews", mimesAliases: {…} }
id: "camerarawpreviews"
​mimesAliases: Object { "image/x-dcraw": "image/jpeg" }
​​"image/x-dcraw": "image/jpeg"
...
registerHandler https://SERVER/apps/viewer/js/viewer-main.mjs?v=b0d516dd-4:2187
    <anonymous> https://SERVER/apps/camerarawpreviews/js/register-viewer.js?v=b0d516dd-4:2

PHP version is 8.2.18 (Debian stable)

helloha82 commented 3 months ago

Which php are you running?

Version: 8.1.28

Memory limit: 10 GB

Max execution time: 3600

Upload max size: 9,8 GB

OPcache Revalidate Frequency: 2

Extensions: Core, date, libxml, openssl, pcre, zlib, filter, hash, json, Reflection, SPL, session, standard, sodium, cgi-fcgi, mysqlnd, PDO, xml, bcmath, calendar, ctype, curl, dom, mbstring, FFI, fileinfo, ftp, gd, gettext, gmp, iconv, igbinary, imagick, intl, exif, mysqli, pdo_mysql, Phar, posix, readline, redis, shmop, SimpleXML, sockets, sysvmsg, sysvsem, sysvshm, tokenizer, xmlreader, xmlwriter, xsl, zip, Zend OPcache

ariselseng commented 3 months ago

I'm also seeing some issue with nextcloud 29.0.1 + camerarawpreviews-0.8.5, firstly there's a JS error in console:

The handler doesn't have a valid component 
Object { id: "camerarawpreviews", mimesAliases: {…} }
id: "camerarawpreviews"
​mimesAliases: Object { "image/x-dcraw": "image/jpeg" }
​​"image/x-dcraw": "image/jpeg"
...
registerHandler https://SERVER/apps/viewer/js/viewer-main.mjs?v=b0d516dd-4:2187
    <anonymous> https://SERVER/apps/camerarawpreviews/js/register-viewer.js?v=b0d516dd-4:2

PHP version is 8.2.18 (Debian stable)

This has been broken in Nextcloud Viewer for years now. The problem is upstream. There is pending PR's which never merges.

ariselseng commented 3 months ago

Which php are you running?

Version: 8.1.28

Memory limit: 10 GB

Max execution time: 3600

Upload max size: 9,8 GB

OPcache Revalidate Frequency: 2

Extensions: Core, date, libxml, openssl, pcre, zlib, filter, hash, json, Reflection, SPL, session, standard, sodium, cgi-fcgi, mysqlnd, PDO, xml, bcmath, calendar, ctype, curl, dom, mbstring, FFI, fileinfo, ftp, gd, gettext, gmp, iconv, igbinary, imagick, intl, exif, mysqli, pdo_mysql, Phar, posix, readline, redis, shmop, SimpleXML, sockets, sysvmsg, sysvsem, sysvshm, tokenizer, xmlreader, xmlwriter, xsl, zip, Zend OPcache

Seems okay. Are you talking about actual previews ( small thumbnails in the file manager) or are you talking about not able to open raw files with the Viewer app?

helloha82 commented 3 months ago

Which php are you running?

Version: 8.1.28 Memory limit: 10 GB Max execution time: 3600 Upload max size: 9,8 GB OPcache Revalidate Frequency: 2 Extensions: Core, date, libxml, openssl, pcre, zlib, filter, hash, json, Reflection, SPL, session, standard, sodium, cgi-fcgi, mysqlnd, PDO, xml, bcmath, calendar, ctype, curl, dom, mbstring, FFI, fileinfo, ftp, gd, gettext, gmp, iconv, igbinary, imagick, intl, exif, mysqli, pdo_mysql, Phar, posix, readline, redis, shmop, SimpleXML, sockets, sysvmsg, sysvsem, sysvshm, tokenizer, xmlreader, xmlwriter, xsl, zip, Zend OPcache

Seems okay. Are you talking about actual previews ( small thumbnails in the file manager) or are you talking about not able to open raw files with the Viewer app?

The thumbnails are fine, it's when you open an image it just suggest a download instead of a preview.

ariselseng commented 3 months ago

See https://github.com/ariselseng/camerarawpreviews/issues/86 This is not a new issue.