Allow non digits (rebased onto develop)

[atarkowska]

This is the same as gh-5014 but rebased onto develop.

---

What this PR does

This PR clean up console.logs and allow non digits to be passed as initially selected tree elements. see ome/omero-mapr#12

[atarkowska]

--rebased-from #5014

[snoopycrimecop]

Conflicting PR. Removed from build OMERO-DEV-merge-push#558. See the console output for more details.
Possible conflicts:

• PR Spw grid view2 #4924 will-moore 'Spw grid view2'
  • components/tools/OmeroWeb/omeroweb/webclient/templates/webclient/base/base_container.html

[snoopycrimecop]

Conflicting PR. Removed from build OMERO-DEV-merge-push#559. See the console output for more details.
Possible conflicts:

• PR Spw grid view2 #4924 will-moore 'Spw grid view2'
  • components/tools/OmeroWeb/omeroweb/webclient/templates/webclient/base/base_container.html

[atarkowska]

we need to be careful how do we declare vars cc @will-moore

[joshmoore]

Travis status, e.g.

     [exec] _______________ TestShow.test_show_instantiation[show_request4] ________________
     [exec] 
     [exec] self = <test_show.TestShow object at 0x4bdf150>
     [exec] show_request = {'initially_select': (), 'request': <WSGIRequest: GET '/webclient?show=illegal-1'>}
     [exec] 
     [exec]     def test_show_instantiation(self, show_request):
     [exec]         show = Show(None, show_request['request'], None)
     [exec] >       assert show.initially_open is None
     [exec] E       assert [] is None
     [exec] E        +  where [] = <omeroweb.webclient.show.Show object at 0x4bdf390>.initially_open
     [exec] 
     [exec] test/unit/test_show.py:109: AssertionError
     [exec]  generated xml file: /home/travis/build/openmicroscopy/openmicroscopy/components/tools/OmeroWeb/target/reports/junit-results.xml 

[will-moore]

Works fine, robot tests and "show" tests passing.

Good to merge.