Skip to content

Fix copy/pasted item implicits#72

Merged
LocalIdentity merged 3 commits into
PathOfBuildingCommunity:masterfrom
Quotae:3.9-implicit-fix
Dec 20, 2019
Merged

Fix copy/pasted item implicits#72
LocalIdentity merged 3 commits into
PathOfBuildingCommunity:masterfrom
Quotae:3.9-implicit-fix

Conversation

@Quotae
Copy link
Copy Markdown
Contributor

@Quotae Quotae commented Dec 16, 2019

Fixes implicits not working on copy/pasted items. Basically just makes it check for (implicit) the same way it checks for (crafted).

Look behind the tooltip and you can see that (implicit) is there and ignored:

image

Copy link
Copy Markdown
Contributor

@NickRyder NickRyder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one small error.

thanks!

Comment thread Classes/Item.lua Outdated
local fractured = line:match("{fractured}") or line:match(" %(fractured%)")
local rangeSpec = line:match("{range:([%d.]+)}")
local crafted = line:match("{crafted}") or line:match(" %(crafted%)")
local implicit = line:match("{implicit}") or line:match(" %(crafed%)")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line:match(" %(crafed%)")

Fixed "crafed" to "implicit" not sure how that happened!!!!!
Copy link
Copy Markdown
Contributor

@NickRyder NickRyder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@LocalIdentity LocalIdentity merged commit 6ecf79e into PathOfBuildingCommunity:master Dec 20, 2019
@Quotae Quotae deleted the 3.9-implicit-fix branch July 15, 2020 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants