Не удаляет данные из бд?

Question

[Nikita]

Описание:

Дело в том , что код работает и ошибок никаких не выдает, но не удаляет почему то.Хотя выскакивает allert , что "название" удалена. Даже после перезапуска бота все ровно данные остаются . Код ниже:

@dp.callback_query_handler (lambda x: x.data and x.data.startswith ('del'))
async def callbeck_run (callbeck_query: types.CallbackQuery):
    await sqlite_db.sql_delete_command (callbeck_query.data.replace ('del', ''))
    await callbeck_query.answer (text=f' {callbeck_query.data.replace ("del" , "")} удалена. ' , show_alert=True )

async def sql_delete_command(data):
    cur.execute ('DELETE FROM memorycfcb WHERE name = %s', (data,))

Comments

[Сергей Горностаев]

Не на Heroku случаем работает?

[Nikita]

Сергей Горностаев, Да там , возможно это потому что я не сделал привзяку по id к каждой записи и не делал первичный ключ? То есть у меня вот так:
photo TEXT NOT NULL ,
name TEXT NOT NULL,
description TEXT NOT NULL
Может стоило добавить id serial primary key или проблема не в этом?

[Сергей Горностаев]

Nikita, на Heroku эфимерная файловая система, ничего сохраняться не будет.

Answer 1

[mayton2019]

Надо проверить следующее.
1) Вот эта штука возвращает число удаленных строк

cur.execute ('DELETE FROM memorycfcb WHERE name = %s', (data,))

проверь что там хотя-бы не ноль. Вообще - всегда проверяй коды состояний API.

Любого API. И зимой и летом проверяй. Это просто как чистить зубы по утрам.

2) Проверь что режим autocommit включен.

Comments

[Сергей Горностаев]

Оно ему вернëт успешные удаления, но изменения в файле данных не сохранятся, SQLite на Heroku бесполезен.

[Nikita]

Сергей Горностаев, Postgres , до деплоя была sqlite

[Алан Гибизов]

Nikita, сам я не пробовал, но замечал, что на heroku у postgres не включен autocommit. Делайте после каждого изменения бд явный commit.

[Сергей Горностаев]

Nikita, кстати, а откуда в функции sql_delete_command берëтся cur?

[Nikita]

Сергей Горностаев, cur=base.cursor,base = ну само подключение к бд, это курсор он там и должен быть.
Курсор,метод экзекьют вроде все верно , но из бд не удаляет, через пг админ в запроснике смотрел , там вообще если этот код написать ошибка ,лол

[Nikita]

Алан Гибизов, Эт делаю

[mayton2019]

Nikita, ты как ловкий тролль всех запутал.

Сколько строк у тебя execute удалил? Посчитал?

[Сергей Горностаев]

Nikita, курсор стоит открывать перед каждым запросом и закрывать сразу после, он не предназначен для повторного использования. А соединение, если оно не управляется пулом, не должно передаваться за пределы контекста выполнения, оно непотокобезопасно. Проблемы могут быть связаны с этими фактами.

[Nikita]

mayton2019, честно говоря не понял о чем ты ) , ну вообще 0 так как ничего не удалил , в тг визуально он мне пишет запись удалена , а по факту нет)

[Nikita]

Сергей Горностаев, понял вас)

Answer 2

[sergeyfilippov4]

не коммитнул

Comments

[Nikita]

И автокоммит пробовал и просто комитет под каждым действием - ничего)