Merge pull request #24 from rlaveycal/issue_23

Only create string_split if DB level doesn't have it
betr-io · Nov 15, 2021 · d520670 · d520670
2 parents 6bff314 + b9be10a
commit d520670
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/sql/user.go b/sql/user.go
@@ -120,7 +120,7 @@ func (c *Connector) CreateUser(ctx context.Context, database string, user *model
                               'DEFAULT_LANGUAGE = ' + Coalesce(QuoteName(@language), 'NONE')
                 END
             END
-          IF objectproperty(object_id('String_Split'), 'isProcedure') IS NULL
+          IF exists (select compatibility_level FROM sys.databases where name = db_name() and compatibility_level < 130)
           BEGIN
               DECLARE @sql NVARCHAR(MAX);
               SET @sql = N'Create FUNCTION [dbo].[String_Split]
@@ -191,7 +191,7 @@ func (c *Connector) UpdateUser(ctx context.Context, database string, user *model
             BEGIN
               SET @stmt = @stmt + ', DEFAULT_LANGUAGE = ' + Coalesce(QuoteName(@language), 'NONE')
             END
-          IF objectproperty(object_id('String_Split'), 'isProcedure') IS NULL
+          IF exists (select compatibility_level FROM sys.databases where name = db_name() and compatibility_level < 130)
           BEGIN
               DECLARE @sql NVARCHAR(MAX);
               SET @sql = N'Create FUNCTION [dbo].[String_Split]