EquipInfo

[cristimirt]

You have to use avatar.EndInspect(), or else the Inspect window will be visible everytime you change targets. Which can be annoying. What you can do is check if the avatar is already inspecting the target. I think the function is avatar.IsTargetInspected(). If it's already inspected, you won't have to call avatar.StartInspect() or avatar.EndInspect(). If the target is not inspected, nothing changes. This could be an example:

local inspecting = avatar.IsTargetInspected()
if not inspecting then avatar.StartInspect() end
{the code to get info about gems goes here}
if not inspecting then avatar.EndInspect() end

There shouldn't be problems. If inspecting is false, your addon starts and ends the inspect like it does in the actual version. If inspecting is true, your addon will not call StartInspect() and EndInspect(), but it will still get info about gems.

[logg]

почти 3х мегабайтный png - зло

Держи jpg

[synmais]

Однако, есть неприятный момент.

Я думаю, что это скорее всего из-за конфликтования с каким-то другим аддоном, у меня не получилось добиться самопроизвольного открытия окна экипировки персонажа. Но я понимаю, что заставлять что-то отключать довольно глупо и я постараюсь решить проблему.

You have to use avatar.EndInspect(), or else the Inspect window will be visible everytime you change targets...

Got it, thx=)

почти 3х мегабайтный png - зло

Держи jpg

Сяп, заменил тут.

[synmais]

Немного изменил алгоритм инспектирования. Замените свой GemsInfo.lua на этот: GemsInfo.lua.zip. Если при смене таргета при открытом окне с информацией по самоцветам будет самопроизвольно открываться окно с осмотром персонажа, то попробуйте временно отключить все аддоны, кроме этого. Если проблема исчезнет, то попробуйте найти конфликтный аддон и сообщите мне. Скорее всего это те аддоны, что показывают какую-либо информацию по другим игрокам. Мне данный баг воспроизвести не удалось, все работает как и положено.

[logg]

if avatar.GetTarget() and avatar.GetTarget() ~= avatar.GetId() and wtMainPanel:IsVisible() then

логичнее делать вначале проверку wtMainPanel:IsVisible()

а не вызывать лишний раз апи функции, т.е.

if  wtMainPanel:IsVisible() and avatar.GetTarget() and avatar.GetTarget() ~= avatar.GetId() then

if GetTableSize( arrEquipment ) > 0 then

имхо проще

if arrEquipment then

[Lyushen]

не стану покупать

[synmais]

if wtMainPanel:IsVisible() and avatar.GetTarget() and avatar.GetTarget() ~= avatar.GetId() then

Согласен, спс.

if GetTableSize( arrEquipment ) > 0 then

имхо проще

if arrEquipment then

Пробовал. Видать {} не равно nil и в таком виде условие пропускало пустую таблицу дальше. Поэтому и пришлось так извратиться.

[logg]

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

for _, id in arrEquipment do
local parentInfo = avatar.GetItemInfo( id )
local enchants = enchant.GetEnchants( id )
if enchants then

и почему то только одна проверка на enchants, а на parentInfo нету

может в этом проблема)

[logg]

Немного изменил алгоритм инспектирования. Замените свой GemsInfo.lua на этот: GemsInfo.lua.zip. Если при смене таргета при открытом окне с информацией по самоцветам будет самопроизвольно открываться окно с осмотром персонажа, то попробуйте временно отключить все аддоны, кроме этого. Если проблема исчезнет, то попробуйте найти конфликтный аддон и сообщите мне. Скорее всего это те аддоны, что показывают какую-либо информацию по другим игрокам. Мне данный баг воспроизвести не удалось, все работает как и положено.

конфликт с NewTarget3DPvP

[synmais]

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

[synmais]

конфликт с NewTarget3DPvP

Ок, спс. Скачаю, гляну что там.

[logg]

Ок, спс. Скачаю, гляну что там.

Да и так понятно, там идет задержка на avatar.StartInspect, поэтому окно и открывается. Над написать Procgun, чтобы тоже добавил 2 переменные на проверку, а не одну.

[synmais]

Первый шаг в сторону локализации аддона на другие языки. Скопировать с заменой в корневую папку аддона Scripts+Locales.zip. В первую очередь интересует, нет ли ошибок в английском варианте фраз (см. Locales/Locales.lua), ну и хотелось бы, чтобы языков было побольше, может есть энтузиасты)) Также может кто-то может перевести описание аддона на англ.? Не все, а хотя бы основные моменты.

[cristimirt]

I think I'll be the one to test the addon (and probably every other addon) on the English servers and also provide localization/fixes if needed

[cristimirt]

Just put the lastest files you provided. I got this error:

Error: addon GemsInfo: Error while running the chunk
Error: addon GemsInfo: [string "Mods/Addons/GemsInfo/Scripts/GemsInfo.lua"]:385: bad argument #1 to `len' (string expected, got nil)
Error: addon GemsInfo: func: ?, ?, line: -1, defined: C, line: -1, [C]
Error: addon GemsInfo:	 func: len, field, line: -1, defined: C, line: -1, [C]
Error: addon GemsInfo:	 func: ?, ?, line: 385, defined: Lua, line: 381, [string "Mods/Addons/GemsInfo/Scripts/GemsInfo.lua"]
Error: addon GemsInfo:		 func: Init, local, line: 506, defined: Lua, line: 505, [string "Mods/Addons/GemsInfo/Scripts/GemsInfo.lua"]
Error: addon GemsInfo:		 func: ?, ?, line: 525, defined: main, line: 0, [string "Mods/Addons/GemsInfo/Scripts/GemsInfo.lua"]
Error: addon GemsInfo: Can't execute script for addon: Mods/Addons/GemsInfo/AddonDesc.(UIAddon).xdb

This is because the eng localization is not finished. You use this in the L function:

if Locales[ localization ] then

Instead, you should use this:

if Locales[ localization ] [ text] then

I'll add english localization now and upload the Locales.lua

[cristimirt]

Here's the english localization:Locales.rar

[cristimirt]

Also, here is the English translation for the description:

A simple addon that shows, in a different window, all the glyphs and their upgrade progress, both yours and your targets.

Attention! With r7 you have the posibility to see the glyphs of your target. Compatibility has been checked so far only with InspectEquipRating and Targeter. If there is a problem when used together with TargetsManager, NewTarget3DPvP, (New)InspectStatsVelhuru or any other addon that inspects players, then write us, and we will investigate.

Are you tired of opening the Character Window, open the Glyph window for each gear item and then hover over each Glyph to see their upgrade progress? Then this addon is for you! The addon has 4 types of sorting:

-the upgrade progress on glyphs

-the quality of the glyphs

-the name of the glyphs

-the gear piece where glyphs are inserted in

LMB on gear item glyphs or progress bar for sorting (see above). By clicking a second time, the sort changes the order)

Shift+LMB on glyphs will sorth the list by the glyphs name

RMB on a glyph will put the glyph in your bag.

*LMB - Left Mouse Button, RMB - Right Mouse Button

[synmais]

I think I'll be the one to test the addon (and probably every other addon) on the English servers and also provide localization/fixes if needed

Here's the english localization:Locales.rar

Also, here is the English translation for the description:

Thank you very much for this

[dudevg]

Из-за аддона может открываться окно осмотра персонажа при кликанье на цель?

Изменено 23 апреля, 2013 пользователем zheka8585

[synmais]

Из-за аддона может открываться окно осмотра персонажа при кликанье на цель?

Тут дело в том, что NewTarget3DPvP пытается обрабатывать события, посылаемые другими аддонами ( GemsInfo, InspectEquipRating ), поэтому и открывается нежелательное окно.

[dudevg]

Тут дело в том, что NewTarget3DPvP пытается обрабатывать события, посылаемые другими аддонами ( GemsInfo, InspectEquipRating ), поэтому и открывается нежелательное окно.

у меня не стоит NewTarget3DPvP, вот в чем дело.

[synmais]

Тогда отключите все другие аддоны, оставив GemsInfo, а затем включайте по одному, чтобы узнать, с каким из них конфликт.

[Василий]

вот тут мне кажется интересно сделан просмотр камней. скрин сверху-справа.

http://alloder.pro/files/file/140-targetparty/

[synmais]

Да, интересно, согласен=) Но для наших целей подходит чуть хуже. Напомню, изначально аддон позиционировался как помощник в крафте, т.е. отсортировав по прогрессу крафта или виду камней можно легко понять что крафтить и куда летать. В том варианте глазками придется пробежаться по всем, попутно считая, где какой % был больше, сколько каких камней уже есть, чтобы летать в тот сектор, откуда нужно больше камней.

Но если народ считает это полезным, могу сделать нечто подобное=)

[Василий]

ну хз. мне показалось довольно наглядно =)