(PYL-E1133) Non-iterable value used in an iterating context

[Dhruvacube]

Description

A non-iterable value is being used in an iterating context, For example, a non-iterable value being passed into a for loop. This will raise a TypeError.

Occurrences

There is 1 occurrence of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/The-4th-Hokage/yondaime-hokage/issue/PYL-E1133/occurrences/

[Pagasis]

Hi,
https://github.com/The-4th-Hokage/yondaime-hokage/blob/08c20fec8ad28353559291da69c505e3d313b184/minato_namikaze/bot_files/lib/mendeleev/models.py#L601-L601
Here self.oxistates isn't defined anywhere before in the file!

[Dhruvacube]

oxistates

@Pagasis it is defined, check this https://github.com/The-4th-Hokage/yondaime-hokage/blob/08c20fec8ad28353559291da69c505e3d313b184/minato_namikaze/bot_files/lib/mendeleev/models.py#L235

its functional attr

[Pagasis]

Oh sorry, yeah so self.oxistates is returning a list, but
https://github.com/The-4th-Hokage/yondaime-hokage/blob/08c20fec8ad28353559291da69c505e3d313b184/minato_namikaze/bot_files/lib/mendeleev/models.py#L235-L239
what is the data type of self._oxidation_states , as you are looping it inside the function oxistates. Like maybe it is causing the issue!

[Dhruvacube]

ohhhh wait @Pagasis sorry for the inconvenience caused but its a false positive issue, actually the issue was raised using the deepsource and I didn't see it, so sorry

[Dhruvacube]

_oxidation_states

actually _oxidation_states is a foreign key object

[Pagasis]

ohhhh wait @Pagasis sorry for the inconvenience caused but its a false positive issue, actually the issue was raised using the deepsource and I didn't see it, so sorry

No problem 😄