jinzhu/gorm

Similar blocks of code found in 2 locations. Consider refactoring.

Open

func (s mysql) HasColumn(tableName string, columnName string) bool {
    currentDatabase, tableName := currentDatabaseAndTable(&s, tableName)
    if rows, err := s.db.Query(fmt.Sprintf("SHOW COLUMNS FROM `%s` FROM `%s` WHERE Field = ?", tableName, currentDatabase), columnName); err != nil {
        panic(err)
    } else {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

func (s mysql) HasIndex(tableName string, indexName string) bool {
    currentDatabase, tableName := currentDatabaseAndTable(&s, tableName)
    if rows, err := s.db.Query(fmt.Sprintf("SHOW INDEXES FROM `%s` FROM `%s` WHERE Key_name = ?", tableName, currentDatabase), indexName); err != nil {
        panic(err)
    } else {

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        if relationship.PolymorphicType != "" {
                            scope.Err(newScope.SetColumn(relationship.PolymorphicType, relationship.PolymorphicValue))
                        }

Avoid deeply nested control flow statements.

Open

                        if joinTableHandler := relationship.JoinTableHandler; joinTableHandler != nil {
                            scope.Err(joinTableHandler.Add(joinTableHandler, newDB, scope.Value, newScope.Value))
                        }

Avoid deeply nested control flow statements.

Open

            } else if reflectValue.Type().Elem().AssignableTo(fieldType.Elem()) {
                // if field's type is slice of struct, then need to set value back to argument after save
                setSliceFieldBackToValue = true
                field.Set(reflect.Append(field.Field, reflectValue.Elem()))
            }

Avoid deeply nested control flow statements.

Open

                    if cp.before == "" {
                        cp.before = c.name
                    }

Avoid deeply nested control flow statements.

Open

                        if err == ErrUnaddressable {
                            results[field.DBName] = value
                        } else {
                            results[field.DBName] = field.Field.Interface()
                        }

Avoid deeply nested control flow statements.

Open

                    if primaryValue, err := result.LastInsertId(); scope.Err(err) == nil {
                        scope.Err(primaryField.Set(primaryValue))
                    }

Avoid deeply nested control flow statements.

Open

                    if primaryValue, err := result.LastInsertId(); scope.Err(err) == nil {
                        scope.Err(primaryField.Set(primaryValue))
                    }

Avoid deeply nested control flow statements.

Open

                        if !field.IsForeignKey || !field.IsBlank || !field.HasDefaultValue {
                            sqls = append(sqls, fmt.Sprintf("%v = %v", scope.Quote(field.DBName), scope.AddToVars(field.Field.Interface())))
                        }

Avoid deeply nested control flow statements.

Open

                    if indirectType.Kind() == reflect.Struct {
                        for i := 0; i < indirectType.NumField(); i++ {
                            for key, value := range parseTagSetting(indirectType.Field(i).Tag) {
                                if _, ok := field.TagSettingsGet(key); !ok {
                                    field.TagSettingsSet(key, value)

Avoid deeply nested control flow statements.

Open

                    } else if relationship := field.Relationship; relationship != nil && relationship.Kind == "belongs_to" {
                        for _, foreignKey := range relationship.ForeignDBNames {
                            if foreignField, ok := scope.FieldByName(foreignKey); ok && !scope.changeableField(foreignField) {
                                sqls = append(sqls,
                                    fmt.Sprintf("%v = %v", scope.Quote(foreignField.DBName), scope.AddToVars(foreignField.Field.Interface())))

Avoid deeply nested control flow statements.

Open

                    if isPtr {
                        results.Set(reflect.Append(results, elem.Addr()))
                    } else {
                        results.Set(reflect.Append(results, elem))
                    }

Avoid deeply nested control flow statements.

Open

                if valuer, ok := v.(driver.Valuer); ok {
                    if v, err = valuer.Value(); err == nil {
                        err = scanner.Scan(v)
                    }
                } else {

Avoid deeply nested control flow statements.

Open

                } else if result.CanAddr() && resultsMap[result.Addr()] != true {
                    resultsMap[result.Addr()] = true
                    results = reflect.Append(results, result.Addr())
                }

Avoid deeply nested control flow statements.

Open

                        if autoCreate {
                            scope.Err(newDB.Save(elem).Error)
                        }

Avoid deeply nested control flow statements.

Open

                    } else if autoUpdate {
                        scope.Err(newDB.Save(elem).Error)
                    }

Avoid deeply nested control flow statements.

Open

                    } else if !field.IsPrimaryKey || !field.IsBlank {
                        columns = append(columns, scope.Quote(field.DBName))
                        placeholders = append(placeholders, scope.AddToVars(field.Field.Interface()))
                    }

Avoid deeply nested control flow statements.

Open

                        if foreignField, ok := scope.FieldByName(foreignKey); ok && !scope.changeableField(foreignField) {
                            columns = append(columns, scope.Quote(foreignField.DBName))
                            placeholders = append(placeholders, scope.AddToVars(foreignField.Field.Interface()))
                        }

Avoid deeply nested control flow statements.

Open

                } else if _, isTime := fieldValue.(*time.Time); isTime {
                    // is time
                    field.IsNormal = true
                } else if _, ok := field.TagSettingsGet("EMBEDDED"); ok || fieldStruct.Anonymous {
                    // is embedded struct