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

[core] : Invalid FK still present in .sql files #17647

Closed
jcastro-dotcms opened this issue Nov 21, 2019 · 4 comments
Closed

[core] : Invalid FK still present in .sql files #17647

jcastro-dotcms opened this issue Nov 21, 2019 · 4 comments
Labels
dotCMS : Content Management dotCMS : Push Publishing Merged QA : Passed Internal Release : 5.2.4 Type : Defect

Comments

@jcastro-dotcms
Copy link
Contributor

jcastro-dotcms commented Nov 21, 2019 •
edited

Related to #13373
Reported via Support: 99850
Reproducible in dotCMS 4.3.3 through latest master (5.2.2).

As per #13373 , the FK_workflow_task_asset foreign key was removed from dotCMS via the following Upgrade Task: https://github.com/dotCMS/core/blob/master/dotCMS/src/main/java/com/dotmarketing/startup/runonce/Task04235RemoveFKFromWorkflowTaskTable.java

However, it's still present in the .sql scripts, e.g., https://github.com/dotCMS/core/blob/master/dotCMS/src/main/resources/postgres.sql#L2251

It must be removed as several customers are still bumping into that problem.

Expected behavior

The FK should not exist anymore.

Acceptance Criteria

  1. Unit test
@jcastro-dotcms
Copy link
Contributor Author

Reported by another customer: https://dotcms.zendesk.com/agent/tickets/100005

jcastro-dotcms added a commit that referenced this issue Dec 13, 2019
@jcastro-dotcms
#17647 : Removing FK that is no longer valid.
b55a4a0
@wezell wezell added this to the Bug Sprint milestone Dec 13, 2019
@jcastro-dotcms
Copy link
Contributor Author

PR: #17742

@wezell
Copy link
Contributor

wezell commented Dec 13, 2019

Adding to our bug sprint - thanks Jose for the PR!

jgambarios pushed a commit that referenced this issue Jan 6, 2020
@jcastro-dotcms @jgambarios
#17647 : Removing FK that is no longer valid. (#17742)
e80e1e7
jgambarios pushed a commit that referenced this issue Jan 6, 2020
@jcastro-dotcms @jgambarios
#17647 : Removing FK that is no longer valid. (#17742)
67950a3 
(cherry picked from commit e80e1e7)
@wezell wezell closed this as completed Jan 29, 2020
@bryanboza
Copy link
Member

Fixed.

@wezell wezell removed this from the Bug Sprint milestone Feb 4, 2020
@wezell wezell mentioned this issue May 25, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dotCMS : Content Management dotCMS : Push Publishing Merged QA : Passed Internal Release : 5.2.4 Type : Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@jgambarios @wezell @dsilvam @bryanboza @erickgonzalez @jcastro-dotcms