Should the forms table contain the user ID rather than user name in the event the user name is changed? If the user name was somehow allowed to be changed it would seem that any SQL that retrieves form data based on user name would be omitted with the name change.
This may be a non-issue as I attempted to change a user name (on my dev box) logged in as Admin and was not able to. It seems that even an Admin cannot change the user name once created. I just thought I would bring this up.
Indeed this is a weakness of the data stored in OpenEMR. The authenticated user name is stamped on records in many tables instead of the unique ID associated with it. Your example in the forms table is just one such place.
Fortunately there are other good examples, such as the postcalendar_events table where the user ID is referenced.
Could you post a feature request into the Tracker system here on SourceForge detailing the request to use User ID and not User Name in the tables?