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

ID field in 40F_SEO_RedirectLog? #1

Open
EPTamminga opened this issue Sep 12, 2019 · 1 comment
Open

ID field in 40F_SEO_RedirectLog? #1

EPTamminga opened this issue Sep 12, 2019 · 1 comment
Assignees
@skamphuis
Labels
enhancement New feature or request

Comments

@EPTamminga
Copy link

What is the need/use of the field with the name ID in the table 40F_SEO_RedirectLog?

It is not a key, naming is not standard, the table is just a log table and does not need a PK? Or if the ID is the PK, shouldn't have been named 40F_SEO_RedirectLogId or similar?
@Timo-Breumelhof Timo-Breumelhof added the question Further information is requested label Oct 11, 2019
@skamphuis skamphuis self-assigned this Nov 8, 2019
@skamphuis
Copy link
Contributor

The Id field is indeed the Primary Key, but the PK constraint is missing. Personally I prefer Primary Key fields to be called Id.

@skamphuis skamphuis added enhancement New feature or request and removed question Further information is requested labels Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skamphuis skamphuis

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@skamphuis @EPTamminga @Timo-Breumelhof