Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collaborative tags stopped working #12810

Closed
codesmaker opened this issue Dec 4, 2018 · 31 comments
Closed

Collaborative tags stopped working #12810

codesmaker opened this issue Dec 4, 2018 · 31 comments
Labels
1. to develop Accepted and waiting to be taken care of bug feature: tags regression

Comments

@codesmaker
Copy link

Hi,

When I create a file or folder and go to details to assign tags, the tags field does not show. That is, when I click on "Tags" nothing happens and for the files and folders that are already tagged, I get the tags field but previously assigned tags aren't showing.

I've recently upgraded from v13 to v14.0.4. It seems that it is a bug in the version that I'm running.

Nextcloud v14.0.4
Collaborative tags v1.4.0

nextcloud-tags2
nextcloud-tags1

@codesmaker
Copy link
Author

Maybe related to #10774 ? For me, it never works though.

Is there any quick workaround to fix it till v14.0.5 is released?

@ChristophWurst ChristophWurst added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap feature: tags labels Dec 5, 2018
@ChristophWurst
Copy link
Member

cc @nickvergessen

@codesmaker
Copy link
Author

Any updates? It is important because file access control depends on it.

@ChristophWurst
Copy link
Member

Tested and works on 15 RC3. I see this is Vue multiselect now, @skjnldsv did you work on this?

@skjnldsv
Copy link
Member

skjnldsv commented Dec 7, 2018

@ChristophWurst not that I remember 🤔

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

Also here no longer works with v14.0.4. With v14.0.3 it was working!

@ChristophWurst ChristophWurst added regression 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Dec 7, 2018
@ChristophWurst ChristophWurst added this to the Nextcloud 14.0.5 milestone Dec 7, 2018
@nickvergessen
Copy link
Member

Just tested again and it works fine here with stable14 which has no change to 14.0.4 in the systemtags app (which is the collaborative tags)

Please press F12 go to "Console" and see if there are any errors while loading the page or trying to open the tags menu

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

This is what I can see in the console after "RELOAD":

Content Security Policy: Die Direktive 'child-src' sollte nicht mehr verwendet werden. Bitte verwenden Sie stattdessen die Direktive 'worker-src' zum Kontrollieren von Workern bzw. die Direktive 'frame-src' zum Kontrollieren von Frames.
JQMIGRATE: Migrate is installed, version 1.4.0
core.js:7:542
unreachable code after return statement[Weitere Informationen]
merged.js:1501:4
window.controllers/Controllers sollte nicht mehr verwendet werden. Verwenden Sie es nicht für die Browser-Erkennung.
merged.js:2176
Source-Map-Fehler: SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data
Ressourcen-Adresse: https://XXX.YYY.ZZZ/nc/apps/notifications/js/notifications.js?v=9bc60c60-5
Source-Map-Adresse: notifications.js.map[Weitere Informationen]
New search handler registered
search.js:66:4
querying Whats New data was successful: nocontent
merged-template-prepend.js:4488:4
undefined
merged-template-prepend.js:4489:4

@nickvergessen
Copy link
Member

Can you disable the notifications app and see if that solves it?

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

Now (notifications disabled) I receive:
Content Security Policy: Die Direktive 'frame-src' sollte nicht mehr verwendet werden. Bitte verwenden Sie stattdessen die Direktive 'child-src'. (unbekannt)
JQMIGRATE: Migrate is installed, version 1.4.0 core.js:7:542
unreachable code after return statement[Weitere Informationen] merged.js:1501:4
New search handler registered search.js:66:4
querying Whats New data was successful: nocontent merged-template-prepend.js:4488:4
undefined merged-template-prepend.js:4489:4

line 1501: function inFileList($row, result) {
return false;

			if (! self.fileAppLoaded()) {
				return false;
			}
			var dir = self.fileList.getCurrentDirectory().replace(/\/+$/,'');
			var resultDir = OC.dirname(result.path);
			return dir === resultDir && self.fileList.inList(result.name);
		}

@nickvergessen
Copy link
Member

I have those warnings too, but do the tags work now?

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

No... .sorry.. .forgot to mention that!

@nickvergessen
Copy link
Member

Can you send me a test account from your instance? Just email it to <my github username>@nextcloud.com

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

Done!

@nickvergessen
Copy link
Member

Strange, the click event is not bound to the item:

events: {
'click': 'click'
},

@skjnldsv do you want to have a look, I tried to step with the JS debugger through it, but it does the exact same steps it's doing in my stable14. Could it be that another app is interfering with Backbone?

@rubbergh can you for testing disable all apps that are not shipped by default? as far as I can see its only dropit and groupfolders, from which none should be causing issues.

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

Am I safe to disable "group folders" and wont loose my data in the group folders?

@rubbergh
Copy link

rubbergh commented Dec 7, 2018

Welll.. also the "favorites star" does not work in detail dialog... should work, or am I wrong?

Well... the star does work if I use the context dialog to active item as favorite... then the star in the detail dialog works again... but only this way.

@sgofferj
Copy link

I don't have dropit installed and disabled group folders for testing. No change here. I see the same problem since the update from .3 to .4.
I also see another symptom which could be related... The PDF preview in the properties sidebar looks messed up and the "loading" animation never stops for PDFs.

@rubbergh
Copy link

Same here also.

@nickvergessen
Copy link
Member

Yeah it seems like something is breaking all JS at some point. Everything that is linked before works (details sidebar can be opened) but everything that would be done after the breaking point does not, e.g. tagging and pdf previews

@rubbergh
Copy link

Do we need to downgrade?

@codesmaker
Copy link
Author

I concur that I have the same PDF preview issue as well.

@rubbergh
Copy link

Should upgrade on 15.0 help us?

@aventrax
Copy link

Exactly same issue here. A few days before I updated my installation from 14.0.3 to 14.0.4 and I enabled the group folders. Tags worked before.

@hlecorche
Copy link

Fixed by #12577 / #12598

@nickvergessen
Copy link
Member

Okay, so it will be fixed with 14.0.5 which is planned for January.

You can copy teh fix from the linked PRs manually for the time being.

@rubbergh
Copy link

Thnx. With patch seems to work again.

@vijaydwivedics
Copy link

Can any give proper solution for this issue?

Collaborative tags stopped working in version 14.0.4
image

@ChristophWurst
Copy link
Member

Can any give proper solution for this issue?

Did you actually read any of the previous comments before you posted yours? #12810 (comment)

@vijaydwivedics
Copy link

Can any give proper solution for this issue?

Did you actually read any of the previous comments before you posted yours? #12810 (comment)

yes i have read but i don't understand what is the solution.

@ChristophWurst
Copy link
Member

No worries.

The fixes are linked at #12810 (comment). The patch release will be available soonish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug feature: tags regression
Projects
None yet
Development

No branches or pull requests

9 participants