Как исправить ошибку в Sequelize: error: duplicate key value violates unique constraint «SmetaCheckupPlans_pkey»?

Question

[GNG999]

Есть модели
//SmetaCheckupPlan

'use strict';
const {
    Model
} = require('sequelize');
module.exports = (sequelize, DataTypes) => {
    class SmetaCheckupPlan extends Model {
        /**
         * Helper method for defining associations.
         * This method is not a part of Sequelize lifecycle.
         * The `models/index` file will call this method automatically.
         */
        static associate(models) {
            // define association here
            SmetaCheckupPlan.belongsTo(models.Smeta, {
                foreignKey: 'smetaId',
                onDelete: 'CASCADE'
            })
        }
    }
    SmetaCheckupPlan.init({
        kind: DataTypes.TEXT,
        supplier:  DataTypes.TEXT,
        address: DataTypes.TEXT,
        phone:DataTypes.TEXT,
        price: DataTypes.TEXT,
    }, {
        sequelize,
        modelName: 'SmetaCheckupPlan',
    });
    return SmetaCheckupPlan;
};

//CheckupPlan

'use strict';
const {
  Model
} = require('sequelize');
module.exports = (sequelize, DataTypes) => {
  class CheckupPlan extends Model {
    /**
     * Helper method for defining associations.
     * This method is not a part of Sequelize lifecycle.
     * The `models/index` file will call this method automatically.
     */
    static associate(models) {
      // define association here
      CheckupPlan.belongsTo(models.Application, {
        foreignKey: 'applicationId',
        onDelete: 'CASCADE'
      })
    }
  }
  CheckupPlan.init({
    kind: DataTypes.TEXT,
    place: DataTypes.TEXT,
    target: DataTypes.TEXT,
    supplier:  DataTypes.TEXT,
    address: DataTypes.TEXT,
    phone:DataTypes.TEXT,
    price: DataTypes.TEXT,
  }, {
    sequelize,
    modelName: 'CheckupPlan',
  });
  return CheckupPlan;
};

Их применяю в след функции

async updateappl(req, res, next) { 
//обновление заявки (заявка это модель )
        try {
         
            const applicationsData = await Application.findOne({where: {id}});
            const [smetaData, created] = await Smeta.findOrCreate({
                where: {id}, defaults: {
                    mostProblDiagnosis,
                    patientName,
                    patientBirthDate,
                    patientPromoter,
                    applicationId: id,
                    creationDate: new Date().toString(),
                    execDate: '',
                }
            });

            if (!created) {
                await smetaData.update({mostProblDiagnosis, patientName, patientBirthDate, patientPromoter})
            }
           
            await CheckupPlan.destroy({where: {applicationId: id}});
            await SmetaCheckupPlan.destroy({where: {smetaId: id}});
            for (const plan of checkupPlans) {
                const result = await CheckupPlan.create({...plan});
                const smetaResult = await SmetaCheckupPlan.create({...plan})
                await result.setApplication(applicationsData);
                await smetaResult.setApplication(smetaData)
            }
            return res.json(applicationsData);
        } catch (e) {
            next(e);
        }
    }

мне нужно при обновлении заявки создать сущность смета и в ее поле SmetaCheckupPlan передать те же данные что и в поле CheckupPlan заявки
у меня выходит ошибка

original: error: duplicate key value violates unique constraint "SmetaCheckupPlans_pkey"

Это потому что я одни те же данные использую ? (. const result = await CheckupPlan.create({...plan});
const smetaResult = await SmetaCheckupPlan.create({...plan}) )

Answer 1

[szQocks]

удали таблицы в бд, и запусти сервер создав таблицы заново - это сработает при условии если проект ещё только только создаётся, если проект уже создан и в будущем нельзя удалить таблицы, делай миграции

Comments

[GNG999]

те вот. так

await CheckupPlan.destroy({where: {applicationId: id}});
            await SmetaCheckupPlan.destroy({where: {smetaId: id}});

Делать нельзя ?

[szQocks]

GNG999, конечно можно, просто у тебя таблицы в бд не соответствуют моделям в коде

[GNG999]

Есть существующий проект , туда нужно добавить новые таблицы : Smeta , SmetaCheckupPlans , где у сметы много таких планов (has many)
Я вручную создал файлы моделей в папке models , и написал сам код внутри моделей . Выше в коде в вопросе там есть .
в главном файле идет sequelize sync alter true я думал этого хватает
Я неправильно сделал ?

[szQocks]

GNG999,

Есть существующий проект , туда нужно добавить новые таблицы

- нужно делать миграции, либо удалять таблицы вообще все и создавать их заного - либо он так и будет выплёвывать ошибку подобную о том что бы поля не совпадают, сам sequelize может их создать, но они конечно потеряют данные которые там были + я не вижу что бы была транзакция, что будет если например CheckupPlan.destroy({where: {applicationId: id}}); удалиться а SmetaCheckupPlan.destroy({where: {smetaId: id}}); - нет ? нужна целостность данных, изучай транзакции

[szQocks]

GNG999,

в главном файле идет sequelize sync alter true я думал этого хватает

- как видишь не хватает, да и не будешь же ты на проде это юзать

[GNG999]

szQocks, нужно делать миграции - те создавать новые модели через миграции ?

[szQocks]

GNG999, дополнять/изменять/удалять/создавать - таблицы через миграции - да

[GNG999]

Понял , спасибо , щас как раз глянул в файлах проекта - там как раз есть миграции

[szQocks]

GNG999, если мало ли непонятно там будет, то скажу короче - суть такая что тебе нужно таблицы в бд, подогнать под модели, сделать это на проде через sync alter true не вариант, так как таблицы очищаются, поэтому подгонять нужно через миграции

вообще по факту можно и без миграций подогнать таблицы с столбцы в бд под модели, тупо вручную но так обычно никто не делает, да и тут шанс большой что, что-то пойдёт не так)

[GNG999]

szQocks,
вроде разобрался, но щас другая ошибка
крч
миграция

'use strict';

module.exports = {
  async up (queryInterface, Sequelize) {
      await queryInterface.createTable('Smetas', {
          id: {
              allowNull: false,
              autoIncrement: true,
              primaryKey: true,
              type: Sequelize.INTEGER
          },
          patientName: {
              allowNull: true,
              type: Sequelize.STRING
          },
          patientBirthDate: {
              type: Sequelize.STRING,
              allowNull: true,
          },
          patientPromoter: {
              type: Sequelize.STRING,
              allowNull: true,
          },
          createdAt: {
              allowNull: true,
              type: Sequelize.DATE
          },
          updatedAt: {
              allowNull: true,
              type: Sequelize.DATE
          }
      });
  },

  async down (queryInterface, Sequelize) {
      await queryInterface.dropTable('Smetas');
  }
};

миграция

'use strict';
module.exports = {
    async up(queryInterface, Sequelize) {
        await queryInterface.createTable('SmetaCheckupPlans', {
            id: {
                allowNull: false,
                autoIncrement: true,
                primaryKey: true,
                type: Sequelize.INTEGER
            },
            kind: {
                type: Sequelize.TEXT
            },
            supplier: {
                type: Sequelize.TEXT
            },
            address: {
                type: Sequelize.TEXT
            },
            phone: {
                type: Sequelize.TEXT
            },
            price: {
                type: Sequelize.TEXT
            },
           smetaId: {
                type: Sequelize.INTEGER,
                onDelete: 'CASCADE',
                references: {
                    model: 'Smetas',
                    key: 'id',
                    as: 'smetaId',
                }
            },
            createdAt: {
                allowNull: false,
                type: Sequelize.DATE
            },
            updatedAt: {
                allowNull: false,
                type: Sequelize.DATE
            }
        });
    },
    async down(queryInterface, Sequelize) {
        await queryInterface.dropTable('SmetaCheckupPlans');
    }
};

Прогнал миграции, сверил с терминалом все создалось
В папке models вручную прописал след модели
//файл smeta.js

'use strict';
const {
    Model
} = require('sequelize');
module.exports = (sequelize, DataTypes) => {
    class Smeta extends Model {
        /**
         * Helper method for defining associations.
         * This method is not a part of Sequelize lifecycle.
         * The `models/index` file will call this method automatically.
         */
        static associate(models) {
            // define association here
            console.log('Associating Smeta with SmetaCheckupPlan');
            Smeta.hasMany(models.SmetaCheckupPlan, {
                foreignKey: 'smetaId',
                onDelete: 'cascade'
            })
            Smeta.hasMany(models.AdditionalCost, {
                foreignKey: 'smetaId',
                onDelete: 'cascade'
            })
        }
    }

    Smeta.init({
        patientPromoter: DataTypes.STRING,
        patientName: DataTypes.STRING,
        patientBirthDate: DataTypes.STRING,
        applId:DataTypes.STRING, // это я сделал доп миграцией addColumn и тоже строку вручную добавил
        execDate: DataTypes.STRING
    }, {
        sequelize,
        modelName: 'Smeta',
        tableName:'Smetas'
    });
    return Smeta;
};

// файл smetacheckupplan.js

'use strict';
const {
    Model
} = require('sequelize');
module.exports = (sequelize, DataTypes) => {
    class SmetaCheckupPlan extends Model {
        static associate(models) {
            console.log('Associating SmetaCheckupPlan with Smeta');
            // define association here
            SmetaCheckupPlan.belongsTo(models.Smeta, {
                foreignKey: 'smetaId',
                onDelete: 'CASCADE'
            })
        }
    }
    SmetaCheckupPlan.init({
        kind: DataTypes.TEXT,
        supplier: DataTypes.TEXT,
        address: DataTypes.TEXT,
        phone:DataTypes.TEXT,
        price: DataTypes.TEXT,
    }, {
        sequelize,
        modelName: 'SmetaCheckupPlan',
    });
    return SmetaCheckupPlan;
};

в контроллере

result = await SmetaCheckupPlan.create({...plan});
                    await result.setSmeta(smetaData);

Выдает ошибку TypeError: result.setSmeta is not a function

[szQocks]

GNG999,

Выдает ошибку TypeError: result.setSmeta is not a function

это хорошо, это уже относиться к связям, если посмотришь на свои связи между таблицами, то всё логично что у тебя sequelize не создал этот метод, отмечай решением и разбирайся со связями) это уже другой вопрос к тому же

SmetaCheckupPlan.belongsTo(models.Smeta, {
                foreignKey: 'smetaId',
                onDelete: 'CASCADE'
            })

если бы тут было не belongsTo а hasMany, тогда бы был метод

[szQocks]

GNG999,

Прогнал миграции, сверил с терминалом все создалось

и сверяться тебе нужно с таблицами в бд, а не с терминалом вхаха)

смотреть что там с таблица в бд получилось в результате, и соответствуют ли они моделям, ну это так на будущее

[GNG999]

szQocks,
под терминалом имел ввиду вход в бд: psql
и далее просмотр таблиц колонок

А так вчера ночью уже в общем разобрался , создаю получаю объект и его ассоциации