From ce1b20789ca5a312166cf327c86daf3e14c969ee Mon Sep 17 00:00:00 2001
From: Jim Lambert <jimlambrt@Jims-MBP.home>
Date: Thu, 13 Feb 2020 21:31:49 -0500
Subject: [PATCH] handle string primary fields

---
 callback_create.go | 24 ++++++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/callback_create.go b/callback_create.go
index 1d64dd30..d64ce374 100644
--- a/callback_create.go
+++ b/callback_create.go
@@ -3,6 +3,7 @@ package gorm
 import (
 	"database/sql"
 	"fmt"
+	"reflect"
 	"strings"
 )
 
@@ -146,12 +147,27 @@ func createCallback(scope *Scope) {
 		}
 
 		if scope.IsOracle() {
-			var id uint32
-			scope.SQLVars = append(scope.SQLVars, sql.Out{Dest: &id})
-			scope.SQL = fmt.Sprintf("%s returning id into :%d", scope.SQL, len(scope.SQLVars))
+			var stringId string
+			var intId uint32
+			primaryIsString := false
+			out := sql.Out{
+				Dest: &intId,
+			}
+			if primaryField.Field.Kind() == reflect.String {
+				out = sql.Out{
+					Dest: &stringId,
+				}
+				primaryIsString = true
+			}
+			scope.SQLVars = append(scope.SQLVars, out)
+			scope.SQL = fmt.Sprintf("%s returning %s into :%d", scope.SQL, scope.Quote(primaryField.DBName), len(scope.SQLVars))
 			if result, err := scope.SQLDB().Exec(scope.SQL, scope.SQLVars...); scope.Err(err) == nil {
 				scope.db.RowsAffected, _ = result.RowsAffected()
-				scope.Err(primaryField.Set(id))
+				if primaryIsString {
+					scope.Err(primaryField.Set(stringId))
+				} else {
+					scope.Err(primaryField.Set(intId))
+				}
 			}
 			return
 		}