PathOfBuilding icon indicating copy to clipboard operation
PathOfBuilding copied to clipboard

Fix kalandra's touch skipping mods

Open Paliak opened this issue 1 year ago • 0 comments

Fixes https://github.com/PathOfBuildingCommunity/PathOfBuilding/issues/8250

Description of the problem being solved:

When filtering out mods of type SocketedIn the logic modified the table causing the iterator to get out of sync and skip mods.

This pr additionally fixes an issue where Kalandra's touch was not taking item overrides into consideration showing stat difference only for the copied ring.

Link to a build that showcases this PR:

eNrtXW1z2siy_hz_ChVVe7-E2NLoPdfeU9hgm6ztEMBxki-pQRpAtpCIXuyQrf3vt3skQGAkJMPWrZzyqS0HSf02z_T09Ix6dI7_83PiCo8sCB3fO6lJh2JNYJ7l2443Oqnd9s_fGbX__Hlw3KHR-OPwNHZcfPLnwZtj_ltw2SNzgU8EvogGIxZ9nsuSv8O9KfWiMfO9a3rvBxe-fVK78T1WEwbUs51ofmW5NAxv6ISd1Joxc50wqgk0tJhnny2f9Fw6YwEQj2lArYgFV6i7EUf-tW_D8yF1Q5A1oY7X860HFl0EfjwF22rCo8OeEqL2dedjt5-xy_GydkHD3hx3uKJeRCMhhD8ntQbgQ0esSSfwF6RRNwZRkn6o61rtqJDnNA7CqBJjb8qYvaQ9VCWSR9oJWGs4ZFbkPLKzwInOxtSzlnrEPL6qtNexGzlT10H0U3pyqKp5LJfPpCtKHm3fj6jb7PSWgrVDQ9fl_EYnHH603fRTF5DMypYPRaKLJNeYOycaP2Mi5qEpKrKsFXG1R54TsUrt6PhO6HtZFvFQNEVDEvVtPKvuRA51uci254rANlMzJEL0rSivwidJuiEr-VxnsetCeMgy6YeKaeiGphl5TF0WsuCRRs4aGrlK_MnA8dZ6FtXIpizn9u019eiZH5ZwG6TssACiT1SJoccsHwJWVR0VOa-cIStPWakdKUNVa17WjlavLF1lwS8zqAtjqhxlz4_dkpTRMmLmD9Im-7mM93IuWduLMmQkX9qjj2Npu3V8jLcuO8swTSQVpiZDhBgkyWZuFBrPQsei7jX96UziCYT7Pn1gXkYM0fMdZjSOPIgRecw6MXIVnzsBy-MjqqnkRwzXfhHfmPphHqMEMS1_eDveJWQ4DcuKIVeZLZhUqXAoLa1SZLUgQbDeI3XbsxYMml4k99YLeJDNJBZFGpClC0MHU5iBy0rzLNWkQ3AJligWKxsxL9W4BEuTD-UirivGrPEFQN2lUSaTOQTvVUWTKEZR9F06XL5pHGkkziJdGNM3AF0gf5WjAmbIuBkzieRjhlw5mOUGU48Fo1lv7DDXrkY9t-yMTkvESQQ6y10K8FV1m9AoxVoRkjsa2OVmk6o2PdIwG7eJWQxXQr4SBPKzHwZpLXDYrGze3wn8e1xZuNXYGsHEj4OSPZ4QZ1uQn5vPJ50k-e0yO7ZWZrn8_HyxRDp1YVm41hCylQssdd1NrPkYRBG1Hpq-PSoNG1dSiWPVvl48nUIcQYdYE1A8m0Lu7WQSmXeauJ36I7hzqVGN0255BUvq0goWycSaFkUtyVG-KZgJrLclX02GvLSKRYdeQ7yYwETANwKu_exegCTldg-svEqtnjjh6rouPwb4T2D8GLd1wmrUkDQteyPXlIB5v2br8qVS5FkFZn6UtyH9gtGwrqM0xyY1fWcC0TQMmzSigp1m259p4FAvIny7KWQ0sMZX0Pvn1HUHEAxOatm7_IpvUZ07bsSCJtxDpQjuukRpjsjxEd9xw1_tydQPIoH9xH86NIhm890vTsjvgJwwcjy-rIaQ5Lo1oTf2nxr2I2rq-74bLrbM6HTKPHtFRj9gTKDzAGOhEbzxeCFMaAhWzxKPBTF_K5ALi3VJNxXyT_1vWRJls05g2Q8XBJYqSl1RRREuFF3U64ooi_hE1nVTrasiIQZcqZoKV7Ipqco_K5t_bZtj6vnQIIjXhGhaXSWyXpc1YkggDRTUwQBJwTtEqhPI_eqKrEgy3NA1pa5qBEUDuV7XZKJKdUkhQKMphm5wJrWu6CBVkYgO1CBP1SXTrKMCsBczfbDTNOCvYYIGsF8BgaKpYTN1U-cqgElR4IZioimKIgIiBkBRl2VsKMCgg9GSodQ1SdVlsFmRDTBIAwrABJqiEaDQJLgFgGmmVieyCKolQxTNuiSpZl0xTRBLdEkFOqIRMELTQaeuqRreloBEJYYMIBgAtaQiLIpC4LcJlhDTlOW6SURQoMsGmCwa0BIFuTXJkBXgINBMxeDchq6L0DT8DbCpKtyRRcCNyKAn6WWJIIYSAei4dKAEyJHXFKW6KkP3A2iGiohgK7BHAH_TUI166hrElCSQCeABdiLYDtRoiIhIS4oOYrhvQEKvwV9TAWGpw2koXiGKAkpM7CVN1GWVdwE85GI0QAh6AIEhxAQIZBMdQJSARwJywMAkRAQMDAOaAD4g1U0FOhYckbdGFUGKAkYrJkABhoJHJa6rqSqiAa0AOtAOZpkymiIpwMN9HZ4BvKKh6SAasMcogFsSNJg1Vl3cc2CERjDqMvvnRE23xpFA4QPwzfFt94r_eDOOomn4_ujo6enpcEqjsT9kPyFdO7T8ydEUmGDovgsfHNd9h2KPGvC_09GnU1saaOeXdzfxt1-tR0Zvhp9G4Y9vdx3v5tG7H3yXW7-Gt9_vtJvRp0tLJMZd48Z8NO8u7p346xRSKOXdl657PpRH9O68N56cXkSybjcvm0_jgUgubuKWrvfPjevebHTu06770BobV-HZ9PGx7w2_DBrjqa43729vLeP7j9ljQ7FFvdchcVe-8LrtJ-Orr0eB5vU-X9mqbARdWf81kB--Xz8oT8PL_s-HJ_ODZH23VUZb5Ou7b_1vH_r9v9jQ7I5Za3Y--vhDguEg9r_2zn81A-W2I925s3ce1fvGh1g8G3X09rWp3Upfvhqe90vvqFJPbk9umh86otH_9lfjdPDl1D8dKw9a57scv-u4T_eNR-P-ujW5Pr_69u7-260IGMYB6xsAws31pHNlt8Svd4_jfuvmw8d778t17-vTyQnvqKN5Tx0nryPCo-QKM4LAgWiWdOPiWrCh0_s0inxf8IdCNGZCn4bUdXyhZ_kxJB6OxWN5EB2RJv7tYe-24WY47_Dw6JoGDzcQKw-n3iiJmug8PGzW0oCexO025tFZabdtfis8ansXdAIOA_NEGAXU7cNE5B51Eg04MSRGhkepefB8wNLHpxdccdK0N8ofguNZAaMhs4VELTZtBlm_gIaGCQJHcwi2I_KXE9FHKsAiLHD8oBwkNl8yrAIia4qydzwS4_LhULNwzBc0QrKiqYxEF9gewDXG8K9XDojsUgFCzHM3wXl176gkhuaj8pb8ge_0QI4QQXvSlYzAjX0pOPtyE5gxVPnVT9b9hM-tr46SQUTTFenVT575Cea__xV-8m-gg6n_6yDKAkJgMfA6ip4lb7iuenWUlZFDDPJ7O0o_fvDRSRCMie-NWEk4enynKHzuJYr-LyT4qZElMeFvGV4MyL7WOnxL5HWtkx0tOpFfR8vqaME9md96vOzLOyTFlMXf2zsagT8Gp4idamhYvmtv3BvAvc69I7IwsiQo-D7s_9s3VNMkxmvkWM3GcI_6NXIku0W_-bSyvxgqqtrrwmV9qBj_LbtE-08_8KXXb56uP419Wslf5mWF8XTDzjO-o_0XknY0shCOpaskdfIC_HfpRJXx-EzdZPxgSdmA0aC0j3COtGZo0xY0vvbcOzKJuQWjSPpDOFtAw8ufhMTGlzrMvwYQf3n-ewC0vOQv_44xhFq8tgNV448bP0qe4c35xTGHJRRCP4gu2CQ8nTU7vXNEZe00RFqwgdQ9FiW1KFmek1oUxAxrTIY0dvH-pxhWtVg_ImbvXiWnujw_mCzKvEHU2H_CeJpI7M-mWJDRuLpKnjTcKBWG6ubFJEnRSGqQ4NjLapakTfjzjLpWgkfbm8aR4PHjXhMntL4P4uEQD26BiijgR9Ja5-ets377cyutxsmy8Lfd3714MkDfSv5dFhT1GK8YFMJ4ECY_T2qfHfbEDWmyiDpuWBNgUeDSKYTHRXUMGp22wAW-AmmcCoLH_IDOZllLgnxJrZ8siKC9MN9YgcNy7Vo832JUohALUbF6KU8anqrKF5R49xkNo6RQNgcpfnwtXwqeJ8ttDj4s4G1PptTN1Zw-3YJEhG4Lo84ZOhYWfhV3OTp5QlWAy6LIP7e_05rMfBn81FmegORhPnMyceVxp08LUOVn13JRTZ7mszeZRXPbnjzMZ17UXvsewJQnZUFVIOkmPRcHg6bhuFg_mduzLZctSPIFfoRJK0hL3PIkXUOMmpMUDhyYRuIofxhnKAqw4gdEchDCZ_msyYGHnDbgs4JItHICIAfQLE2-qKRyPjeQFbEmtbS5-KWVuQVdkJal58CfPC0AYV6an9P-9HHBIOHxt_HoO3ZSnZ0zXNbIigIGJBy7i-El57uLWa9B313iuUvDh9z-Tp_ms99GDmYiG6QkCVApITiodpOAY2s3CX1YrkVxwF4soLueiSx5u8U5yKJoeiPz_GlR5EhrqV8sIan4fjE7L0h_MTeP_002ZLhUKJoAFjQF4yOKvSaAERWMjZKiuFmbA8mydZVkJVPhxpZWlpgM8PSgZFEMSEi2CIK5_LIgWywnaXGu4pJRF0_D--5uAp8dCN1FWLIfQj17Lu7jphx92Q8l0fOjEGTyAwNNPIW2K4Yem8w2CMq36_hovqrjRwlwnZUegOhFAZ5_-OX7k6982Ym_0gWnnC4yIeNuOtBvAfe3uR4k_DI_gHLcjmCZma548fdiwZsvIA5ZciL8jtGp73GOzDIVpfDGyMkiv0thOTJ7L9zetD_dtg7-oi5gGtD_Ad7wf0Oh78fW-KALTnpw6zk_Yia0m-8FU2JDjSkaFaktisPhQCJDSZR021RtUx4wuLSlgTEc6PZAlyA9togi6dSWVXNo2qJ1wK3geLwXDO0AVjWuY8EoeC-IB102xCVomLxV9nH4CtwA3FuAdXKXeiMmBPgXD8KqtcyKG7oERa-2VJJWm9ptdFsHF4EzEVozBs0KYdG-3sKhbciWYogaGdiDgaHQIdVFfWBqsjmQNNHShjYRNW1omAPbJJqItfJMUbWBzkxVHqirLVQO-K8u-_Fe0NRsc8nB31ZAhxGz__mKzT0bO_zrEgA8xPVQGIJf_WICA9-EFbAw9ANBPCRpgX0oPAE5w52nmUADJlAeBpgtDGaCNZc0YHQSHryVcAsnHdZCMukI6QRx8FZV8GnbiyDFcUboTwdvDRXvTVIOzBMO3irqH3hzccJMSJJSjNIHXIQhNGx79cWeMIUOXFF48JZwKYgJ7pwlpePAhZsqS0DeSuvawDUFfuwsozcs9ov07GMBhbyVQtlKoW6l0LZS6FspjI0-jntO-YM-HRxpjb7rRwKsdCedwW33CiNUsqd04fqPOCviI9xtFOem5DEkagSpAsupD_FaaAxmYUjdtMsFtYoA5kbr_KQCP88iBfkFrRR6T3S6rlp5iaAdmp_KIKVk8EdtGwQsC2Tw1NBc2Af2BEMvvZVVtgsi2h4Q0faAyItkbGqQVBJafqZuFdr0VgVoN44PeQ-Qkn3BUcXjLxkk3FEpQCsMd21nC0hlCTurlPeFv7oHZ5D2ML6kPdihVPSEfUVtsuuMpewBwOqzlrqv9su7tp_sOISrGWDPhGTfchcfTLKbXSRsHtjqzhK0fXWrtK8YQ0pOeXj6eHXGS-5UmPASzy47x_KT22vpS3KrgsrNvaDs7Evqfpy6ythqTGKXRXuIxPIeApq8Y1BQduSX9oP_i2fqyvFcqczxkixuR1TVXeeKPXWLsq_gppWMNfwzCauxJr1VfalEdp6GVwJ_BT7c0sr6wPy7OlWDG9lPL-5trtvHoll5WZuqop8ZAPLLwN9TuqTuPMXtPklqVcPJruHnucJk56zHouQbRwG1WY_v198x3GMMk019vunOa7Z8b-iM0u335CLdgOf8iztC5EQuftY7qS1b3XPL1m7hDp2Dbxv4u4ak6sSuCQPfdxn10h38o-d8TSfEb1S2LjtYIPLRS1-Mb-Vb6MOaoy7DLy-6sxJ8ccief1ZqK9cgwK-BYgHVmNl9n7exipGcgW-AlwJllY_XDzlYMrmRseNSi41912ZByo5b6rP0O-TzKrrFZyxzGVa-kTZnk7dxLXaw50Vzc05CFGmbxsx3zOds6hae7FfvFjxbmwZ3XmAf6noBG9_8X3IYSj79ZPFldvzSNwuY3eNFen7sRT3mDjNVkGqJNlYGZtF7HXwRuuj0slzVOwJ9bB1Sw9RKdEQlC9Gz1tXIZbykrJae5fN3odBZ4bNS1VwN6ccvfdflb0Gyo3ML53ySnjNokjn_MuQmntAZOe7HIa-gACN5kCtr5OqXOit5_qLOaqELv9NVgOIYgn0OiMdHi0koeR_Nr_48OD5a__-y-D8L_pzP

Before screenshot:

obraz obraz

After screenshot:

obraz obraz

Paliak avatar Sep 27 '24 17:09 Paliak