Comment crashes on SQL Server if comment already exists

[AndreasSchindler]

Issue Creation Checklist

[x ] I have read the contribution guidelines

Issue Description

Storing Tables to SQL Server crashes if an extended Property already exists.

we suggest changing the Function CommentTemplate in MS SQL Querygenerator to

commentTemplate(comment, table, column) {
// Check if extended Property exists; insert / update accordingly
return IF NOT EXISTS (SELECT NULL FROM SYS.EXTENDED_PROPERTIES WHERE [major_id] = OBJECT_ID(${this.quoteIdentifier(table)}) AND [name] = N'MS_Description' AND [minor_id] = (SELECT [column_id] FROM SYS.COLUMNS WHERE [name] = ${this.escape(column)} AND [object_id] = OBJECT_ID(${this.escape(table)}))) EXECUTE sp_addextendedproperty @name = N'MS_Description', @value = ${this.escape(comment)}, @level0type = N'SCHEMA', @level0name = N'dbo', @level1type = N'TABLE', @level1name = ${this.escape(table)}, @level2type = N'COLUMN', @level2name = ${this.escape(column)}; ELSE EXECUTE sp_updateextendedproperty @name = N'MS_Description', @value = ${this.escape(comment)}, @level0type = N'SCHEMA', @level0name = N'dbo', @level1type = N'TABLE', @level1name = ${this.escape(table)}, @level2type = N'COLUMN', @level2name = ${this.escape(column)};;

 }

Additional context

Add any other context or screenshots about the issue here.

Issue Template Checklist

Is this issue dialect-specific?

• No. This issue is relevant to Sequelize as a whole.
• [ x] Yes. This issue only applies to the following dialect(s): MSSQL
• I don't know.

Would you be willing to resolve this issue by submitting a Pull Request?

YES -> See code to fix above

• [x ] Yes, I have the time and I know how to start.
• Yes, I have the time but I don't know how to start, I would need guidance.
• No, I don't have the time, although I believe I could do it if I had the time...
• No, I don't have the time and I wouldn't even know how to start.

I am a first time contributor and am not 100% sure about the steps expected

[wbourne0]

Can you add a code example that shows this failing? Is this when using .sync?

[github-actions]

This issue has been automatically marked as stale because it has been open for 7 days without activity. It will be closed if no further activity occurs. If this is still an issue, just leave a comment or remove the "stale" label. 🙂

[github-actions]

This issue has been automatically marked as stale because it has been open for 7 days without activity. It will be closed if no further activity occurs. If this is still an issue, just leave a comment or remove the "stale" label. 🙂