search

mrackley / StratusForms

Lightweight InfoPath alternative for SharePoint 2007,2010,2013,2016,2019 and Office 365
82 stars 36 forks source link

StratusFormsLookup stores Label, not the Value #29

Closed davidcfk closed 6 years ago

davidcfk commented 6 years ago

Hello! The StratusFormsLookup stores the display value rather than the option value to StrataFormsData...

Edit: Apologies, loading from ID works fine (tested by manually popping in the value into StratusFormsData), but saving to it is putting the Display text rather than the option value.

davidcfk commented 6 years ago

Hmm well, updated line 860 (b1.5) to stick the value in rather than the text, I wasn't sure how valid the thought of having more than one text associated to a single value (or usually, supposedly unique IDs) to identify options in a dropdown box would be... This poses a considerable problem, especially for dropdown lists operating around Sharepoint List lookups (and having the need to save back to the field) -- was that what you meant when you said you would prefer your users to have a separate text column to house only the text selection?

davidcfk commented 6 years ago

Woops. As it turns out, just have to add SFUseLookupID to the class for it to use the ID.

mrackley commented 6 years ago

Correct... all you have to do is us SFUseLookupID... if that jerk of who maintains the library would update the documentation this would have been more apparent... (that jerk being me)... sorry...