Javascript review on 20231018

[marco-brandizi]

I have done a routine review of KnetMiner code, in preparation for 5.7. Regarding Javascript, we still need to address what follows.

• user-access.js: there are still various issues and code to clean, I've reopened User access management in the client code #768 with comments, have a look there
• gene-table-filtering-draft.js: I had proposed a cleaner design of filtering code in this file. It looks like code remained where it was. At this point, I no longer know if a major overhaul is worth for a legacy release. If we decide not, the current code should be explained in a separated .md document, describing the logics, flow, and examples. That would be very valuable in case we still need to make small changes or debug it.
• data-utils.js: genomeApi() has still a minor issue, see line 18
• web-cache.js: apparently, this remained in the poor state it was before my review, but I ain't it clear if now we're using web-cache-new.js. We need to clean and finalise it, it can't go in a new release, not even a legacy release.

Misc Notes

• data-utils.js, I moved openAccessionNetworkView(). Keep the testing code at the end of file. In general, sort things in a file according to some logical order, eg, more abstract and public stuff, implementation and internal details. Do not throw code in files in random order