WebUI: Remove unnecessary hashing

[Chocobo1]

Now the containers support using string as key so the intermediate hashing/mapping to number isn't needed now.

[thalieht]

Changing a category's save path collapses the categories section in the side filters and can't restore it. Also torrent list context menu doesn't open after this. Refreshing the page fixes them.

[glassez]

Changing a category's save path collapses the categories section in the side filters and can't restore it. Also torrent list context menu doesn't open after this. Refreshing the page fixes them.

@Chocobo1
I hope you won't ruin WebUI with such changes on the eve of v5.1 release.
I mean, it might be worth delaying risky changes.

[Chocobo1]

Changing a category's save path collapses the categories section in the side filters and can't restore it. Also torrent list context menu doesn't open after this. Refreshing the page fixes them.

Thanks! Fixed.

I mean, it might be worth delaying risky changes.

IMO this is only a moderate clean up. I'm more afraid of the diverge of stable branch and master branch that cause backport PRs to fail silently or harder to apply. Also we freeze new features only since RC is released, isn't?

[glassez]

Also we freeze new features only since RC is released, isn't?

Yes. But the state of the codebase needs to be stable enough for RC to be released, so we should be more restrained at this stage of the release cycle.
Hopefully, the above problems were the result of a minor oversight, and not an indicator of the difficult-to-predict consequences of this "moderate clean up".
(Just a word of caution. I didn't delve into it much.)