nextcloud-metadata icon indicating copy to clipboard operation
nextcloud-metadata copied to clipboard

Add Nextcloud 28.x Compatibility.

Open roliverio opened this issue 1 year ago • 23 comments

Quick issue to update the app to support Nextcloud 28.x onwards. Thanks!

roliverio avatar Dec 12 '23 14:12 roliverio

@roliverio thanks for the reminder.

gino0631 avatar Dec 12 '23 14:12 gino0631

@roliverio thanks for the reminder. ping

RainerEmrich avatar Jan 04 '24 16:01 RainerEmrich

Is there an update on this? I really like this app, but it isn't working right now.

Chrispikaan avatar Jan 08 '24 18:01 Chrispikaan

Apologies for the delay, but this time it will take longer as there are more changes in the Nextcloud itself.

gino0631 avatar Jan 08 '24 21:01 gino0631

Any progress on this one?

galandilias avatar Feb 07 '24 10:02 galandilias

@galandilias thanks for the ping. Unfortunately, still no, though I hope to make it this month.

gino0631 avatar Feb 08 '24 22:02 gino0631

@galandilias thanks for the ping. Unfortunately, still no, though I hope to make it this month.

Firstly, the real cause for this issue: OCA.Files.FileList was a private API that app developers should not have relied upon.

And secondly, NextCloud frontend has been gradually migrating to Vue for rendering. Best "resource" I could find about this was this issue: https://github.com/nextcloud/server/issues/20020

plaes avatar Feb 12 '24 17:02 plaes

@plaes the first version of your message named the reasons correctly.

Firstly, the real cause for this issue: OCA.Files.FileList was a private API that app developers should not have relied upon.

This has nothing to do with the issue.

As for the Vue, yes, the support for the non-Vue sidebar apps was silently removed, which is the biggest problem for now.

gino0631 avatar Feb 14 '24 22:02 gino0631

I regularly get these log lines over and over, this probably has to do with compatibility: "message":"Could not boot metadata: Call to undefined method OC\\Server::getEventDispatcher()"

immanuelfodor avatar Feb 19 '24 14:02 immanuelfodor

@immanuelfodor yes, this was already reported in #106

gino0631 avatar Feb 19 '24 15:02 gino0631

The same error message was displayed in another project (OCCWEB). The solution here was the call OC::$server->getEventDispatcher(), through OC::$server->get(\OCP\EventDispatcher\IEventDispatcher::class), to replace. Just for info.

PeterLue avatar Feb 20 '24 12:02 PeterLue

@PeterLue yes, but that is not enough for compatibility with NC 28

gino0631 avatar Feb 21 '24 15:02 gino0631

I regularly get these log lines over and over, this probably has to do with compatibility: "message":"Could not boot metadata: Call to undefined method OC\\Server::getEventDispatcher()"

Hi there, thanks for a great app. Just reporting I get lots of these in the logs with Nextcloud 28. Reading this thread, sounds like work is still ongoing for an update, so thank you. 😄

nooblag avatar Feb 29 '24 12:02 nooblag

@nooblag yes, you can try disabling the app meanwhile

gino0631 avatar Feb 29 '24 12:02 gino0631

Hello, when will nextcloud 28 compatibility come?

aliyasiny65 avatar Mar 04 '24 21:03 aliyasiny65

I regularly get these log lines over and over, this probably has to do with compatibility: "message":"Could not boot metadata: Call to undefined method OC\\Server::getEventDispatcher()"

Hi there, thanks for a great app. Just reporting I get lots of these in the logs with Nextcloud 28. Reading this thread, sounds like work is still ongoing for an update, so thank you. 😄

I having the same output in the logs/issue with the app. I am also waiting for an update :-/

Ch3ck3rM0n avatar Mar 11 '24 12:03 Ch3ck3rM0n

In the meanwhile you can use the Memories App, to display the metadata.

SKB-CGN avatar Mar 17 '24 16:03 SKB-CGN

Hi there,

I noticed no problem with the metadata app on NC 28 since a few month, besides the massive log noise output.

Is it possible to get the app uploaded with PeterLue fix, in the meantime?

Thanks.

yeupou avatar Mar 28 '24 17:03 yeupou

I've just received notice of the close-by release of NC 29.xx and this app hasn't had updates for v28. Do you plan on still maintaining it?. Thanks for any update.

roliverio avatar Mar 28 '24 21:03 roliverio

@roliverio thanks for the ping. No, this app is not abandoned. As for Nextcloud, I'm seriously disappointed, and personally decided just not to upgrade, as it only gets worse with every new release (though I haven't found any suitable replacement so far). The works on the app depend on free time I have, and therefore don't go smoothly sometimes (especially when the workload is higher due to the nature of changes in Nextcloud).

gino0631 avatar Mar 29 '24 01:03 gino0631

@yeupou are you sure the app is working for you on NC 28?

gino0631 avatar Mar 29 '24 01:03 gino0631

@gino0631 I understand and i don't blame you at all. I do love your work, and i appreciate the existence of this app, however, i can't deny that i'm blocked from upgrading because i can't use it on newer versions. So i just wanted to know if you would continue working on it. Thanks again.

roliverio avatar Mar 29 '24 01:03 roliverio

@yeupou are you sure the app is working for you on NC 28?

I set up a NC28 dev environment and made the change suggested by @PeterLue above, but the app isn't working, the Metadata tab doesn't show up in the sidebar. Looks like front-end changes are needed as well, but I'm not familiar enough with Nextcloud app development to take a crack at it.

FWIW, Nextcloud 28 API changes are documented here: https://docs.nextcloud.com/server/28/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_28.html

IndrekHaav avatar Apr 03 '24 09:04 IndrekHaav

Oh yes, pleeease, add 28.x compatibility. Without I will not update to NC 28.x.x because I love your app!

jgrete avatar May 08 '24 08:05 jgrete