character_inventory очищена - Ru-MaNGOS

KiriX · 19.09.2011, 13:31

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

По идее - не беда. Все данные остались в item_inctance. Я полагал, этого хватит и сервер, как обычно всё вернёт игроку служебной почтой.
Однако, этого почему-то не произошло... Записи есть, но сервер вернул всего 5-6 вещей и всё.
Есть варианты, как заставить сервер вернуть все вещи? Или вообще хоть как-то их вернуть

19.09.2011, 16:57

Может быть попробовать создать записи в character_inventory с неверными значениями слотов и сумок?

KiriX · 19.09.2011, 20:14

Цитата:

Сообщение от gmltA

Может быть попробовать создать записи в character_inventory с неверными значениями слотов и сумок?

Может быть... И что туда ставить вместо верных значений???

19.09.2011, 20:34

Цитата:

Сообщение от KiriX

Может быть... И что туда ставить вместо верных значений???

проще гм-ом добавить инвентарь...
зы и не забыть настроить регулярный дамп БД

19.09.2011, 21:24

Цитата:

Сообщение от KiriX

Может быть... И что туда ставить вместо верных значений???

Ввести гуид чара, гуид из item_instance, остальное оставить нулевыми значениями.

KiriX · 19.09.2011, 22:20

Цитата:

Сообщение от Minimajack

проще гм-ом добавить инвентарь...
зы и не забыть настроить регулярный дамп БД

Это не интересно...
+ вещи-то все есть в item_inctance, вот и интересно, как их оттуда можно нормально выудить

Vinolentus · 19.09.2011, 22:25

Код:

DELIMITER $$
CREATE PROCEDURE `RestoreItems`(`char_guid` INT UNSIGNED) DETERMINISTIC
BEGIN
    DECLARE OBJECT_FIELD_ENTRY      INT DEFAULT 3;
    DECLARE MAIL_STATIONERY_GM      INT DEFAULT 61;
    DECLARE MAX_MAIL_ITEMS          INT DEFAULT 12;
    DECLARE MAIL_CHECK_MASK_COPIED  INT DEFAULT 4;
    DECLARE MAIL_NORMAL             INT DEFAULT 0;

    DECLARE mail_id         INT UNSIGNED;
    DECLARE mail_item_idx   INT UNSIGNED;
    DECLARE out_guid        INT UNSIGNED;
    DECLARE out_entry       INT UNSIGNED;
    DECLARE next_row        TINYINT DEFAULT 1;
    DECLARE res CURSOR FOR
        SELECT
            `item_instance`.`guid`,
            SUBSTRING_INDEX(SUBSTRING_INDEX(`item_instance`.`data`, ' ', 4), ' ', -1)
        FROM `item_instance`
        LEFT JOIN `character_inventory` ON `item_instance`.`guid` = `character_inventory`.`item`
        LEFT JOIN `mail_items`          ON `item_instance`.`guid` = `mail_items`.`item_guid`
        WHERE
        (
            `owner_guid` = `char_guid`
            AND `character_inventory`.`item`    IS NULL
            AND `mail_items`.`item_guid`        IS NULL
            AND `auction`.`itemguid`            IS NULL
            AND `guild_bank_item`.`item_guid`   IS NULL
        );
    DECLARE CONTINUE HANDLER FOR NOT FOUND SET next_row = 0;

    SELECT MAX(`id`) INTO mail_id FROM `mail`;
    IF mail_id IS NULL THEN
       SET mail_id = 0;
    END IF;

    OPEN res;
    WHILE next_row != 0 DO
        SET mail_item_idx = 0;
        SET mail_id = mail_id + 1;

        WHILE (mail_item_idx < MAX_MAIL_ITEMS AND next_row) DO
            FETCH res INTO out_guid, out_entry;
            IF next_row != 0 THEN
                INSERT INTO mail_items (`mail_id`, `item_guid`, `item_template`, `receiver`)
                VALUES (mail_id, out_guid, out_entry, char_guid);
                SET mail_item_idx = mail_item_idx + 1;
            END IF;
        END WHILE;
        IF mail_item_idx != 0 THEN
            INSERT INTO mail
            (
            --  0   1            2           3               4       5
                id, messageType, stationery, mailTemplateId, sender, receiver,
            --  6        7     8          9            10            11     12   13
                subject, body, has_items, expire_time, deliver_time, money, cod, checked
            )
            VALUES
            (
            --  0        1            2                   3  4            5
                mail_id, MAIL_NORMAL, MAIL_STATIONERY_GM, 0, char_guid, char_guid,
            --  6        7
                'Items', 'There\'s were problems with equipping item(s).',
            --  8     9           10 11 12 13
                true, 30 * 86400, 0, 0, 0, MAIL_CHECK_MASK_COPIED
            );
        END IF;
    END WHILE;

    CLOSE res;
END$$
DELIMITER ;

KiriX · 19.09.2011, 22:40

Vinolentus, спасибо, попробую

KiriX · 22.09.2011, 13:16

Vinolentus
Отлично! Прокатило!

Только я немного поправил ваш запрос, там некоторые ошибочки были...

Код:

DELIMITER $$
CREATE PROCEDURE `RestoreItems`(`char_guid` INT UNSIGNED) DETERMINISTIC
BEGIN
    DECLARE OBJECT_FIELD_ENTRY      INT DEFAULT 3;
    DECLARE MAIL_STATIONERY_GM      INT DEFAULT 61;
    DECLARE MAX_MAIL_ITEMS          INT DEFAULT 12;
    DECLARE MAIL_CHECK_MASK_COPIED  INT DEFAULT 4;
    DECLARE MAIL_NORMAL             INT DEFAULT 0;

    DECLARE mail_id         INT UNSIGNED;
    DECLARE mail_item_idx   INT UNSIGNED;
    DECLARE out_guid        INT UNSIGNED;
    DECLARE out_entry       INT UNSIGNED;
    DECLARE next_row        TINYINT DEFAULT 1;
    DECLARE res CURSOR FOR
        SELECT
            `item_instance`.`guid`,
            SUBSTRING_INDEX(SUBSTRING_INDEX(`item_instance`.`data`, ' ', 4), ' ', -1)
        FROM `item_instance`
        LEFT JOIN `character_inventory` ON `item_instance`.`guid` = `character_inventory`.`item`
        LEFT JOIN `mail_items`          ON `item_instance`.`guid` = `mail_items`.`item_guid`
	LEFT JOIN `auction`             ON `item_instance`.`guid` = `auction`.`itemguid`
	LEFT JOIN `guild_bank_item`     ON `item_instance`.`guid` = `guild_bank_item`.`item_guid`
        WHERE
        (
            `owner_guid` = `char_guid`
            AND `character_inventory`.`item`    IS NULL
            AND `mail_items`.`item_guid`        IS NULL
            AND `auction`.`itemguid`            IS NULL
            AND `guild_bank_item`.`item_guid`   IS NULL
        );
    DECLARE CONTINUE HANDLER FOR NOT FOUND SET next_row = 0;

    SELECT MAX(`id`) INTO mail_id FROM `mail`;
    IF mail_id IS NULL THEN
       SET mail_id = 0;
    END IF;

    OPEN res;
    WHILE next_row != 0 DO
        SET mail_item_idx = 0;
        SET mail_id = mail_id + 1;

        WHILE (mail_item_idx < MAX_MAIL_ITEMS AND next_row) DO
            FETCH res INTO out_guid, out_entry;
            IF next_row != 0 THEN
                INSERT INTO mail_items (`mail_id`, `item_guid`, `item_template`, `receiver`)
                VALUES (mail_id, out_guid, out_entry, char_guid);
                SET mail_item_idx = mail_item_idx + 1;
            END IF;
        END WHILE;
        IF mail_item_idx != 0 THEN
            INSERT INTO mail
            (
            --  0   1            2           3               4       5
                id, messageType, stationery, mailTemplateId, sender, receiver,
            --  6        7     8          9            10            11     12   13
                subject, body, has_items, expire_time, deliver_time, money, cod, checked
            )
            VALUES
            (
            --  0        1            2                   3  4            5
                mail_id, MAIL_NORMAL, MAIL_STATIONERY_GM, 0, char_guid, char_guid,
            --  6        7
                'Items', 'There\'s were problems with equipping item(s).', -- '
            --  8     9           10 11 12 13
                true, 30 * 86400, 0, 0, 0, MAIL_CHECK_MASK_COPIED
            );
        END IF;
    END WHILE;

    CLOSE res;
END$$
DELIMITER;

НО! Выяснилось, что коррозии подвергся не один перс

Можно ли как-то эту процедуру модифицировать, чтобы она сразу по всем персам пробежалась???

Vinolentus · 22.09.2011, 18:51

Можно. Но не уверен, что это хорошая идея. Когда проверял, обнаружил у себя в item_instance достаточно большое (~1.4%) количество записей, не связанных с другими таблицами. Может быть, только моя ошибка.

KiriX · 23.09.2011, 09:28

Цитата:

Сообщение от Vinolentus

Можно. Но не уверен, что это хорошая идея. Когда проверял, обнаружил у себя в item_instance достаточно большое (~1.4%) количество записей, не связанных с другими таблицами. Может быть, только моя ошибка.

Как я понимаю, используется алгоритм формирования писем как это делает сам сервер?
Тогда мне не понятно, почему же сам сервер не формирует такие письма, ведь функционал-то у него такой есть...
Прверьте, плиз, скриптик:

Код:

DELIMITER $$
CREATE PROCEDURE `RestoreItems`() DETERMINISTIC
BEGIN
    DECLARE OBJECT_FIELD_ENTRY      INT DEFAULT 3;
    DECLARE MAIL_STATIONERY_GM      INT DEFAULT 61;
    DECLARE MAX_MAIL_ITEMS          INT DEFAULT 12;
    DECLARE MAIL_CHECK_MASK_COPIED  INT DEFAULT 4;
    DECLARE MAIL_NORMAL             INT DEFAULT 0;

    DECLARE mail_id         INT UNSIGNED;
    DECLARE mail_item_idx   INT UNSIGNED;
    DECLARE out_guid        INT UNSIGNED;
    DECLARE out_entry       INT UNSIGNED;
    DECLARE next_row        TINYINT DEFAULT 1;
    DECLARE res CURSOR FOR
        SELECT
            `item_instance`.`guid`,
            SUBSTRING_INDEX(SUBSTRING_INDEX(`item_instance`.`data`, ' ', 4), ' ', -1),
			`item_instance`.`owner_guid`
        FROM `item_instance`
        LEFT JOIN `character_inventory` ON `item_instance`.`guid` = `character_inventory`.`item`
        LEFT JOIN `mail_items`          ON `item_instance`.`guid` = `mail_items`.`item_guid`
	LEFT JOIN `auction`             ON `item_instance`.`guid` = `auction`.`itemguid`
	LEFT JOIN `guild_bank_item`     ON `item_instance`.`guid` = `guild_bank_item`.`item_guid`
        WHERE
        (
			`character_inventory`.`item`    	IS NULL
            AND `mail_items`.`item_guid`        IS NULL
            AND `auction`.`itemguid`            IS NULL
            AND `guild_bank_item`.`item_guid`   IS NULL
        );
    DECLARE CONTINUE HANDLER FOR NOT FOUND SET next_row = 0;

    SELECT MAX(`id`) INTO mail_id FROM `mail`;
    IF mail_id IS NULL THEN
       SET mail_id = 0;
    END IF;

    OPEN res;
    WHILE next_row != 0 DO
        SET mail_item_idx = 0;
        SET mail_id = mail_id + 1;

        WHILE (mail_item_idx < MAX_MAIL_ITEMS AND next_row) DO
            FETCH res INTO out_guid, out_entry, char_guid;
            IF next_row != 0 THEN
                INSERT INTO mail_items (`mail_id`, `item_guid`, `item_template`, `receiver`)
                VALUES (mail_id, out_guid, out_entry, char_guid);
                SET mail_item_idx = mail_item_idx + 1;
            END IF;
        END WHILE;
        IF mail_item_idx != 0 THEN
            INSERT INTO mail
            (
            --  0   1            2           3               4       5
                id, messageType, stationery, mailTemplateId, sender, receiver,
            --  6        7     8          9            10            11     12   13
                subject, body, has_items, expire_time, deliver_time, money, cod, checked
            )
            VALUES
            (
            --  0        1            2                   3  4            5
                mail_id, MAIL_NORMAL, MAIL_STATIONERY_GM, 0, char_guid, char_guid,
            --  6        7
                'Items', 'There\'s were problems with equipping item(s).', -- '
            --  8     9           10 11 12 13
                true, 30 * 86400, 0, 0, 0, MAIL_CHECK_MASK_COPIED
            );
        END IF;
    END WHILE;

    CLOSE res;
END$$
DELIMITER;

Так должно сработать по всем персам сразу. Просто процедуры для мускула никогда не писал

KiriX · 23.09.2011, 14:54

Подправил скриптик, проверил, вроде работает, но нужен нормальный тест

Код:

DELIMITER $$
CREATE PROCEDURE `RestoreItems`() DETERMINISTIC
BEGIN
    DECLARE OBJECT_FIELD_ENTRY      INT DEFAULT 3;
    DECLARE MAIL_STATIONERY_GM      INT DEFAULT 61;
    DECLARE MAX_MAIL_ITEMS          INT DEFAULT 12;
    DECLARE MAIL_CHECK_MASK_COPIED  INT DEFAULT 4;
    DECLARE MAIL_NORMAL             INT DEFAULT 0;

    DECLARE mail_id         INT UNSIGNED;
    DECLARE mail_item_idx   INT UNSIGNED;
    DECLARE out_guid        INT UNSIGNED;
    DECLARE out_entry       INT UNSIGNED;
		DECLARE char_guid       INT UNSIGNED;
    DECLARE next_row        TINYINT DEFAULT 1;
    DECLARE res CURSOR FOR
        SELECT
            `item_instance`.`guid`,
            SUBSTRING_INDEX(SUBSTRING_INDEX(`item_instance`.`data`, ' ', 4), ' ', -1),
						`item_instance`.`owner_guid`
        FROM `item_instance`
        LEFT JOIN `character_inventory` ON `item_instance`.`guid` = `character_inventory`.`item`
        LEFT JOIN `mail_items`          ON `item_instance`.`guid` = `mail_items`.`item_guid`
		LEFT JOIN `auction`             ON `item_instance`.`guid` = `auction`.`itemguid`
		LEFT JOIN `guild_bank_item`     ON `item_instance`.`guid` = `guild_bank_item`.`item_guid`
        WHERE
        (
            `item_instance`.`owner_guid` 				<> 0
            AND `character_inventory`.`item`    IS NULL
            AND `mail_items`.`item_guid`        IS NULL
            AND `auction`.`itemguid`            IS NULL
            AND `guild_bank_item`.`item_guid`   IS NULL
        );
    DECLARE CONTINUE HANDLER FOR NOT FOUND SET next_row = 0;

    SELECT MAX(`id`) INTO mail_id FROM `mail`;
    IF mail_id IS NULL THEN
       SET mail_id = 0;
    END IF;

    OPEN res;
    WHILE next_row != 0 DO
        SET mail_item_idx = 0;
        SET mail_id = mail_id + 1;

        WHILE (mail_item_idx < MAX_MAIL_ITEMS AND next_row) DO
            FETCH res INTO out_guid, out_entry, char_guid;
            IF next_row != 0 THEN
                INSERT INTO mail_items (`mail_id`, `item_guid`, `item_template`, `receiver`)
                VALUES (mail_id, out_guid, out_entry, char_guid);
                SET mail_item_idx = mail_item_idx + 1;
            END IF;
        END WHILE;
        IF mail_item_idx != 0 THEN
            INSERT INTO mail
            (
            --  0   1            2           3               4       5
                id, messageType, stationery, mailTemplateId, sender, receiver,
            --  6        7     8          9            10            11     12   13
                subject, body, has_items, expire_time, deliver_time, money, cod, checked
            )
            VALUES
            (
            --  0        1            2                   3  4            5
                mail_id, MAIL_NORMAL, MAIL_STATIONERY_GM, 0, char_guid, char_guid,
            --  6        7
                'Items', 'There\'s were problems with equipping item(s).', -- '
            --  8     9           10 11 12 13
                true, 30 * 86400, 0, 0, 0, MAIL_CHECK_MASK_COPIED
            );
        END IF;
    END WHILE;

    CLOSE res;
END$$
DELIMITER;

KiriX · 25.09.2011, 14:47

Решил сам затестить, стерев полностью таблицу character_inventory. Обнаружил, что мой вариант не работает. Приходят персу чужие шмотки. Нашёл ошибку, но не знаю как побороть:

Код:

        WHILE (mail_item_idx < MAX_MAIL_ITEMS AND next_row) DO
            FETCH res INTO out_guid, out_entry, char_guid;
            IF next_row != 0 THEN
                INSERT INTO mail_items (`mail_id`, `item_guid`, `item_template`, `receiver`)
                VALUES (mail_id, out_guid, out_entry, char_guid);
                SET mail_item_idx = mail_item_idx + 1;
            END IF;
        END WHILE;

Здесь идёт заполнение 12 итемов для письма и не важно кому по факту принадлежит предмет. Просто пихается 12 штук, а потом письмо отправляется игроку, чья запись была найдена/обработана последней в цикле. Не могу понять, как можно сделать проверку, чтобы 12 предметов для письма заполнялись только если эти предметы принадлежат одному игроку, если не принадлежат, выход из цикла и отправка письма, дальше формируется письмо для другого игрока.
Есть идеи?

25.09.2011, 15:35

Кирилл, а вы не думали что проще написать дополнительную процедуру; которая будет пробегаться по всем персонажам и для каждого вызовет восстановление? У вас есть рабочий код, зачем что то в нем менять?

KiriX · 25.09.2011, 16:07

Цитата:

Сообщение от Minimajack

Кирилл, а вы не думали что проще написать дополнительную процедуру; которая будет пробегаться по всем персонажам и для каждого вызовет восстановление? У вас есть рабочий код, зачем что то в нем менять?

Дак я ж писал, не срабатывает он почему-то

Для того перса у которого обнаружилась проблема восстановил несколько предметов и всё. Ждал неделю, перезагружался сервак, процедура не восстанавливала ничего. Так что использовать её же смысла нет. Она не работает.
Если только правда в коде что-то свой написать... Просто тут вроде процедура уже рабочая есть

Думал, может из неё сделать можно

Вариант рабочий - одна шмотка - одно письмо. Но хотелось бы если уж делать, так оптимально

Добавлено через 26 минут

Цитата:

Сообщение от Minimajack

Кирилл, а вы не думали что проще написать дополнительную процедуру; которая будет пробегаться по всем персонажам и для каждого вызовет восстановление? У вас есть рабочий код, зачем что то в нем менять?

АЙ БЛИН!!! Спасибо!

Теперь дошло, что вы имели ввиду

Написать процедуру, которая для каждого перса вызовет рабочую процедуру! Щас попробую

Спасибо!

KiriX · 25.09.2011, 16:27

Если я всё правильно понял, то это должно выглядеть так:

Код:

BEGIN
	DECLARE char_guid INT UNSIGNED;
	DECLARE next_row TINYINT DEFAULT 1;
	DECLARE res CURSOR FOR
		SELECT
			`guid`
		FROM `characters`
	ORDER BY `guid`;
	DECLARE CONTINUE HANDLER FOR NOT FOUND SET next_row = 0;

	OPEN res;
	WHILE next_row != 0 DO
		FETCH res INTO char_guid;
		CALL RestoreItems(char_guid);
	END WHILE;	
	CLOSE res;
END;

Можно ещё оптимизировать сразу отобрав только тех персов, у которых имеются проблемы с вещами таким запросом:

Код:

SELECT
	`item_instance`.`owner_guid`
FROM `item_instance`
LEFT JOIN `character_inventory` ON `item_instance`.`guid` = `character_inventory`.`item`
LEFT JOIN `mail_items`          ON `item_instance`.`guid` = `mail_items`.`item_guid`
LEFT JOIN `auction`             ON `item_instance`.`guid` = `auction`.`itemguid`
LEFT JOIN `guild_bank_item`     ON `item_instance`.`guid` = `guild_bank_item`.`item_guid`
WHERE
(
	`item_instance`.`owner_guid` 		<> 0
	AND `character_inventory`.`item`    IS NULL
	AND `mail_items`.`item_guid`        IS NULL
	AND `auction`.`itemguid`            IS NULL
	AND `guild_bank_item`.`item_guid`   IS NULL
)
GROUP BY `item_instance`.`owner_guid`;

19.09.2011, 13:31	#1
KiriX Умный Старожил Регистрация: 06.03.2010 Сообщений: 886 Сказал(а) спасибо: 698 Поблагодарили 433 раз(а) в 181 сообщениях Записей в дневнике: 4	character_inventory очищена В общем случилась такая беда, что эта таблица оказалась очищена почти полностью для одного игрока. По идее - не беда. Все данные остались в item_inctance. Я полагал, этого хватит и сервер, как обычно всё вернёт игроку служебной почтой. Однако, этого почему-то не произошло... Записи есть, но сервер вернул всего 5-6 вещей и всё. Есть варианты, как заставить сервер вернуть все вещи? Или вообще хоть как-то их вернуть

25.09.2011, 14:47	#13
KiriX Умный Старожил Регистрация: 06.03.2010 Сообщений: 886 Сказал(а) спасибо: 698 Поблагодарили 433 раз(а) в 181 сообщениях Записей в дневнике: 4	Решил сам затестить, стерев полностью таблицу character_inventory. Обнаружил, что мой вариант не работает. Приходят персу чужие шмотки. Нашёл ошибку, но не знаю как побороть: Код: WHILE (mail_item_idx < MAX_MAIL_ITEMS AND next_row) DO FETCH res INTO out_guid, out_entry, char_guid; IF next_row != 0 THEN INSERT INTO mail_items (`mail_id`, `item_guid`, `item_template`, `receiver`) VALUES (mail_id, out_guid, out_entry, char_guid); SET mail_item_idx = mail_item_idx + 1; END IF; END WHILE; Здесь идёт заполнение 12 итемов для письма и не важно кому по факту принадлежит предмет. Просто пихается 12 штук, а потом письмо отправляется игроку, чья запись была найдена/обработана последней в цикле. Не могу понять, как можно сделать проверку, чтобы 12 предметов для письма заполнялись только если эти предметы принадлежат одному игроку, если не принадлежат, выход из цикла и отправка письма, дальше формируется письмо для другого игрока. Есть идеи?

25.09.2011, 15:35	#14
Minimajack Гость Сообщений: n/a	Кирилл, а вы не думали что проще написать дополнительную процедуру; которая будет пробегаться по всем персонажам и для каждого вызовет восстановление? У вас есть рабочий код, зачем что то в нем менять? Последний раз редактировалось Minimajack; 25.09.2011 в 15:36. Причина: сори опечатки в имени =)

19.09.2011, 16:57	#2
gmltA Гость Сообщений: n/a	Может быть попробовать создать записи в character_inventory с неверными значениями слотов и сумок?

19.09.2011, 22:40	#8
KiriX Умный Старожил Регистрация: 06.03.2010 Сообщений: 886 Сказал(а) спасибо: 698 Поблагодарили 433 раз(а) в 181 сообщениях Записей в дневнике: 4	Vinolentus, спасибо, попробую

22.09.2011, 18:51	#10
Vinolentus Пользователь Регистрация: 23.03.2010 Сообщений: 51 Сказал(а) спасибо: 14 Поблагодарили 32 раз(а) в 15 сообщениях	Можно. Но не уверен, что это хорошая идея. Когда проверял, обнаружил у себя в item_instance достаточно большое (~1.4%) количество записей, не связанных с другими таблицами. Может быть, только моя ошибка.