PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Eldritich Mods from imported/copypasted items not being taged correctly and thus not being replaced correctly
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit src/Classes/ImportTab.lua um die Zeilen 834-841 und verfolge den parseRaw-Pfad für kopierten Item-Text. Vergleiche importierte und kopierte Zeilen impliziter Modifikatoren mit den Modifikator-Daten und bestätige, dass übereinstimmende Eldritch-Mods die Tags {eater} oder {exarch} erhalten. Erledigt bedeutet, dass importierte und kopierte Beispiele konsistent getaggt werden und das Ersetzungsverhalten korrigiert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100