SQL Plugin: Added SQL Server compability; Added FK and PK compability for MySQL and SQL Server

[brunomartinspro]

No description provided.

[brunomartinspro]

@davidjgraph @christ66 @alderg Can i humbly ask for feedback about why my comment was deleted and when can this go to production? Thank you
Stackoverflow Question i made for this feature and how it can be tested: https://stackoverflow.com/questions/49198853/foreign-key-on-sql-plugin

[alderg]

Thanks for the PR, this will be merged via our main repo and will appear in the next release.

[alderg]

The plugin is available via Arrange, Insert, From SQL using this link.

[Eddie-Hartman]

@brunomartinspro Did this functionality break? Your examples in the SO post do not create linkages between the tables as foreign keys.
Example:

[brunomartinspro]

@brunomartinspro Did this functionality break? Your examples in the SO post do not create linkages between the tables as foreign keys.
Example:

Unfortunately i never figured out how to visually link the foreign keys in 2018.
The rest remained the same as it was before, even with SQL it didn't linked the boxes to the properties.

Script generated from SSMS:

USE [test]
GO
/****** Object:  Table [dbo].[Rocket]    Script Date: 20/08/2020 10:44:19 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE TABLE [dbo].[Rocket](
	[Id] [int] IDENTITY(1,1) NOT NULL,
	[StatusId] [int] NOT NULL,
 CONSTRAINT [PK_Rocket] PRIMARY KEY CLUSTERED 
(
	[Id] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON, OPTIMIZE_FOR_SEQUENTIAL_KEY = OFF) ON [PRIMARY]
) ON [PRIMARY]
GO
/****** Object:  Table [dbo].[RocketStatus]    Script Date: 20/08/2020 10:44:19 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE TABLE [dbo].[RocketStatus](
	[Id] [int] IDENTITY(1,1) NOT NULL,
	[Name] [nchar](30) NOT NULL,
 CONSTRAINT [PK_RocketStatus] PRIMARY KEY CLUSTERED 
(
	[Id] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON, OPTIMIZE_FOR_SEQUENTIAL_KEY = OFF) ON [PRIMARY]
) ON [PRIMARY]
GO
ALTER TABLE [dbo].[Rocket]  WITH CHECK ADD  CONSTRAINT [FK_Rocket_RocketStatus] FOREIGN KEY([StatusId])
REFERENCES [dbo].[RocketStatus] ([Id])
GO
ALTER TABLE [dbo].[Rocket] CHECK CONSTRAINT [FK_Rocket_RocketStatus]
GO

MySql example copied from Draw.IO: Foreign Key on SQL Plugin :

[gaurakshay]

The following functionalities are not working for me:

1. Identification of the table name.
2. Identification of the foreign or primary key

I have identified that the reason for the table name not populating is that there should be a space between the table name and the starting parenthesis. I don't know why the foreign key and primary key identification is not working.

The result that I get if I use (taken from here ):

CREATE TABLE Persons(
PersonID int,
LastName varchar(255),
FirstName varchar(255),
Address varchar(255),
City varchar(255)
);

CREATE TABLE Orders(
    OrderID int NOT NULL,
    OrderNumber int NOT NULL,
    PersonID int,
    PRIMARY KEY (OrderID),
    FOREIGN KEY (OrderID) REFERENCES Persons(PersonID)
);

[bolehlav]

Same here, Example used there doesn't work for me either.

[brunomartinspro]

@Eddie-Hartman @bolehlav Well something changed... now we have to add the plugin first before we can use it...
Did you see the same UI look as the last picture?

[Eddie-Hartman]

@brunomartinspro I do not have to add it first. I can just go to:

1. Arrange
2. Insert
3. Advanced
4. SQL

Your version is still there if you add the plugin first, I had assumed that the two were based off of the same code though.
They do look different in the UI on creation and the end result though.

[lastlink]

Good news is that I have a pr with improved sql parsing and working foreign relationship lines. See the new pr #3091