Keeping spaces out of content ids in Plone 5

Looking for some comment on this:

  https://dev.plone.org/ticket/13260#comment:47

In Plone 4, portal_skins/plone_scripts/check_id is used on creation
(but unfortunately not on rename, which causes headache for me a few
times a month when users accidentally put spaces in shortname/id).

I am assuming that check_id has not yet been replaced for PLIP 13260,
and I would like to work on this problem.

I would like to implement validation against spaces in shortnames on a
folder's _checkId() method, likely an addition to
plone.app.folder.base.BaseBTreeFolder._checkId() -- which is
(usually?) used by name chooser.  I would also like to have the rename
actions for items and for folder contents us this.

Is this a sane place to put this?  Alternatives?  Any input helpful.

Sean

------------------------------------------------------------------------------
Want fast and easy access to all the code in your enterprise? Index and
search up to 200,000 lines of code with a free copy of Black Duck
Code Sight - the same software that powers the world's largest code
search on Ohloh, the Black Duck Open Hub! Try it now.
http://p.sf.net/sfu/bds