PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Eldritich Mods from imported/copypasted items not being taged correctly and thus not being replaced correctly

オープン
#4,940 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

enhancement good first issue
主要言語
Lua
スター
5.4k
フォーク
2.4k
平均マージ
1日 12時間
マージ済み PR(30日)
26

説明

Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
Is your feature request related to a problem?

Eldritich Mods from imported/copypasted items not being taged correctly and thus not being replaced correctly

Describe the solution you'd like

mods should get the {eater} or {exarch} tags before them

Describe alternatives you've considered

No response

Additional context

For correctly tagging the mods from imported items, you would have to parse that in parseRaw?
from actual import items I guess you do it in https://github.com/PathOfBuildingCommunity/PathOfBuilding/blob/dev/src/Classes/ImportTab.lua#L834-L841 but for copy pasted items that are like "7% Chance to Block Spell Damage (implicit)" and not "{eater} 7% Chance to Block Spell Damage" its probibly done in parseRaw

could do somthing like

local index = nil
for i, implictMod in ipairs(item.implicitModLines) do
      for _, mod in ipairs(modList) do
          for _, modLine in ipairs(mod.mod) do
              if modLine == implictMod.line then
                  index = i
                  break
              end
          end
          if index then
              break
          end
      end
      if index then
          break
       end
end
if index then

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

src/Classes/ImportTab.lua の 834-841 行付近から始め、コピーされたアイテムテキストの parseRaw パスを追跡します。インポートした暗黙的モディファイア行とコピーした暗黙的モディファイア行をモディファイアデータと比較し、一致する Eldritch mods に {eater} または {exarch} タグが付与されることを確認します。完了とは、インポートした例とコピーした例に一貫してタグが付与され、置換の動作が修正されていることを意味します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
lua
領域
backend
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。