Укажите отработанное время

Отработанное время:
Продуктивное время:
Bug 6864 - Переделать реализацию функции OleUIAddVerbMenuA/W   Make a simular bug
Summary: Переделать реализацию функции OleUIAddVerbMenuA/W
Status: CLOSED FIXED
Alias: None
Product: WINE@Etersoft
Classification: Продукты (Products)
Component: OLE / DDE / RPC (show other bugs)
Version: 2.0
Hardware: PC All
: P4 minor
Target Milestone: ---
Assignee: Константин Кондратюк
QA Contact: Денис Баранов
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 42 5101 5618
  Show dependency tree / graph
 
In work:
Reported: 2011-01-28 18:20 MSK by Виталий Перов
Modified: 2011-09-14 14:56 MSK (History)
2 users (show)

See Also:
Заявки RT:
Связано с:
Дата напоминания:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Виталий Перов 2011-01-28 18:20:29 MSK 
Откатил патч:
commit ed1e96915e7dc00f9a3cd2dee271a502daa350dc
Author: Vitaly Lipatov <lav@etersoft.ru>
Date:   Mon Apr 28 08:58:29 2008 +0400

    oledlg: Implement OleUIAddVerbMenuA/W functions

Проблема только в том, что в oledlg_En.rc нет больше таблицы STRINGTABLE DISCARDABLE.
Требуется переделать патч.
Так же хорошо бы вспомнить к какой баге он относится (через поиск найти не удалось)
Comment 1 Виталий Перов 2011-02-15 18:18:31 MSK 
Откатил тот же патч в eterhack. Ещё откатил более поздний патч:

commit bca118976aabace2b2cd27ae72c1438e1431e45b
Author: Vitaly Lipatov <lav@etersoft.ru>
Date:   Sun Jan 4 13:40:57 2009 +0300

    restructured OleUIAddVerbMenuW
Comment 2 Константин Кондратюк 2011-09-14 10:53:54 MSK 
Приступил к переделыванию. Единственная задача по переработке - перенести строки в STRINGTABLE oledlg.rc и текст строк в po-файлы.
Comment 3 Константин Кондратюк 2011-09-14 14:27:31 MSK 
В процессе переделки выяснилось, что vitperov@ откатил не 2 патча, а 4, просто этот факт почему-то не отобразился в ревертах.

Привёл функцию к состоянию, в каком она была после всех патчей. Получилось 2 коммита - один добавляет функцию и ресурсы, а другой включает все изменения этой функции.

Отправил в eterwine.

Локализацию решено сделать позднее.
Comment 4 Константин Кондратюк 2011-09-14 14:56:22 MSK 
Проверено. Меню появляется (но не работает).
Поведение идентично версии 1.0.12 + переведён пункт меню.