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

Orphaned tags. #2270

Closed
Rurik19 opened this issue Sep 7, 2016 · 12 comments · Fixed by #2397
Closed

Orphaned tags. #2270

Rurik19 opened this issue Sep 7, 2016 · 12 comments · Fixed by #2397
Labels

Comments

@Rurik19
Copy link
Contributor

Rurik19 commented Sep 7, 2016

Issue details

After deleting article with unique tag, this tag remains in the tags table, accessible from nowhere.
API and UI the have the same behaviour

@tcitworld tcitworld added the Bug label Sep 8, 2016
@tcitworld tcitworld added this to the 2.1.1 milestone Sep 8, 2016
@nicosomb nicosomb modified the milestones: 2.1.1, 2.1.2 Oct 3, 2016
@j0k3r
Copy link
Member

j0k3r commented Oct 7, 2016

Are you sure about from the web interface?
Because we should remove the tag if no entries are associated: #1663 (see 567421a#diff-ebd626b4b8a67462013f01007d7a4637R70)

@Rurik19
Copy link
Contributor Author

Rurik19 commented Oct 9, 2016

Just checked web interface:

  • add new tag to the article
  • delete article (by clicking on trash icon on article card)
  • i still see the tag in database with phpmyadmin

@Rurik19 Rurik19 reopened this Oct 9, 2016
@j0k3r
Copy link
Member

j0k3r commented Oct 9, 2016

Did you try on the master directly? Because, as the milestone says, it'll be released with the 2.1.2 version which is not yet available.

@Rurik19
Copy link
Contributor Author

Rurik19 commented Oct 10, 2016

Yes, that is on 2.1.1 from master.

@j0k3r
Copy link
Member

j0k3r commented Oct 10, 2016

Master is the upcoming 2.1.2.
Are you sure that tag isn't associated to an other entry?

@Rurik19
Copy link
Contributor Author

Rurik19 commented Oct 10, 2016

Yes, that was "Just_for_test" tag :)

@j0k3r
Copy link
Member

j0k3r commented Oct 10, 2016

Well, it works for me, so..
@nicosomb @tcitworld could you please try on your side?

@nicosomb
Copy link
Member

I have the problem with SQLite too.

@j0k3r
Copy link
Member

j0k3r commented Oct 11, 2016

@Rurik19 could you please tell me step by step all actions to reproduce the bug? I've tried on PostgreSQL, MySQL & SQLite and it works every time.

@Rurik19
Copy link
Contributor Author

Rurik19 commented Oct 12, 2016

@j0k3r
0) PHP Version 5.6.24-0+deb8u1
MySQL ver: 5.5.52-0+deb7u1

  1. git pull origin master
    ➜ wallabag2 git:(4533342) ✗
  2. SYMFONY_ENV=prod composer install -o --prefer-dist
  3. add tag to article
  4. delete article
    image
  5. look at phpmyadmin
    image

@Rurik19 Rurik19 closed this as completed Oct 12, 2016
@Rurik19 Rurik19 reopened this Oct 12, 2016
@nicosomb nicosomb removed this from the 2.1.2 milestone Oct 15, 2016
@nicosomb
Copy link
Member

In fact, what is the problem with this behavior?
The tag is still in database because maybe it's used by an other user (a tag has no relation with a user).
It stays in database but if we affect this tag to a new article, this tag will be used.

@Kdecherf
Copy link
Member

Is this issue still relevant?

@j0k3r j0k3r closed this as completed Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants