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.

Sign up for GitHub

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

Jump to bottom

Bug running migration "Fix user names " #5191

Closed
isaacmg410 opened this issue Jun 12, 2019 · 0 comments · Fixed by #5209
Closed

Bug running migration "Fix user names " #5191

isaacmg410 opened this issue Jun 12, 2019 · 0 comments · Fixed by #5209
Labels
type: bug Issues that describe a bug

Comments

@isaacmg410
Copy link
Contributor

isaacmg410 commented Jun 12, 2019 •
edited

Describe the bug

When you upgrade to the latest version and the migration is added and executed FixUserNames (https://github.com/decidim/decidim/blob/0.18-stable/decidim-core/db/migrate/20190412131728_fix_user_names.rb), it fails because there are users who are "deleted". No email, no name, no nickname, and deleted_at is present.

Issue on MetaDecidim: https://meta.decidim.org/processes/bug-report/f/210/proposals/14638

Expected behavior
Don't crash if there are deleted users, also, 'UserBaseEntity' should has scope ':not_deleted'

Screenshots
If applicable, add screenshots to help explain your problem.

Stacktrace
If applicable, add the error stacktrace to help explain your problem.

Extra data (please complete the following information):

  • Decidim Version: 0.18, master

Additional context
Add any other context about the problem here. For instance, add Metadecidim link.
@isaacmg410 isaacmg410 added type: bug Issues that describe a bug lot: core labels Jun 12, 2019
@oriolgual oriolgual mentioned this issue Jun 18, 2019
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug Issues that describe a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix user names migration decidim/decidim
1 participant
@isaacmg410