Comment 1 for bug 1722827

Revision history for this message
Rogan Hamby (rogan-hamby) wrote :

Hi Jane,

I was looking at this and the core issue is that a space at the beginning or end of a name could theoretically be a valid org unit name. I question why anyone would do that but .... there are more things in heaven and earth than dreamt of in my philosophies.

Still, if we want to trim surrounding spaces from it this should do it:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=blobdiff;f=Open-ILS/src/sql/Pg/030.schema.metabib.sql;h=17b39fbcc380737d01c5f309d4bb2feb6b545589;hp=bc72c896017f793b21623132da28b20f0ec76d3f;hb=d9f0334c29b242f27bf8f6b85381595c7e2719c9;hpb=d6a247f3847fa26e8e5cadf8cd92ff573a828584

/user/rogan/lp1722827_trim_spaces_from8569s

FWIW, I think this is worth doing, barring anyone pointing out use cases it would break and it worked in my (admittedly brief) testing.