GitLab

ensures that the hidden attribute doesn't get overridden by another css
display rule

This was problematic since file2 is not always used and when editing a
file we won't always be selecting a file to upload.

adapting to match what was done in the FormBuilder. This is was easier
given the way the FormBuilder was implemented.

self was undefined causing the serializeObject method to fail silently

the onchange callback used to set the value of the hidden field
"filename" needed altering to fit the multi upload paradigm.

split the actual form part into a partial as with sample_sets, this
allows us to have shorter files with less levels of indentation, making
a little easier to read.

also applies the multi-form naming convention so we can apply
serializeObject to the form.

refactors a couple elements from the controller into functions (such as
validation and pre_process listing) to shorten the controller itself.

working from the form submission was making the task of adapting the
tokeniser to an extendable form more complicated then simply allowing it
to be used on form input.

fixes uses of build_date in SetFormBuilder and subclasses

rename info to build_info and date to build_date

moves some methods up a level and renames some variables to be used in
future FormBuilders that don't inherit from SetFormBuilder

The build method intended for the GenericFormBuilder was assigned to the
SetFormBuilder prototype. Although it didn't cause any functionnal
issues, I feel it needed to be fixed.

make the inputs look a little bit nicer with a little bit of css from
bootstrap

reduces the size of fields in form to save more space.

Although we do have server-side validation (and still need it to prevent
tampering with forms or even the use of outdated browsers),  this will
serve two purposes:
1) alleviate the server slightly
2) have a faster response time for users

so win-win :).

makes better use of space for adding multiple patients/runs/sets in a
single form.

storing the result of the validation in the dict was a bad idea when no
errors are found since we were attempting to insert the error field in
into the database. Of course this column doesn't exist and was causing
an error.

The data attribute 'group-ids' had not been altered to the generic
attribute 'keys'  from the refactor to use with autocompletion on
samples

sets the 'type' argument for redirection after adding sets to the most
abundant type of set.

For ex. when adding mostly patients and some of the other types, the
user will be redirected to the list of patients once the form has been
successfully submitted.

when editing a set, the error key is not present in the patient/run/set,
so we needed to manage that case

When errors are detected in the form, the route was rendering a form
that no longer exists

The permission check in the form route was operating on the sample_set
id rather than the patient/run/set id

sample sets now share a single add/edit route.

moves the new patient/form and patient/submit routes to the sample_set
controller and removes any add/edit routes from all set controllers.
The same goes for the templates.

Also allows the inclusion of different types of sets in a single form.

For this, the sample_set/form.html template includes patials for each
set type.

See #2878

One form to rule them all, one form the find them, one form to bring
them all, and in the template bind them.

Makes use of more different classes to build different forms.

web2py Row objects are compatible with dot notation and getitem
notation, so dot notation is preferable in that case. However, we want
to be able to use dicts built from our json forms, so we need to use
getitem notation everywhere.

makes sure none of the elements from the header overlap into the main
content.

makes the refactored patient form extendable. With the serializeObject
method, form names can be used like array/object accessors and enables
complexe json objects to be built from the form and sent to the
controller.

See #2878

Having a single controller and view, is better for clean code practices,
since changes to the form/controller won't require two forms and
controllers to be changed.

So when the changes for the multi-patient form come in, we'll have less
to do and a cleaner code-bade to work from.

I also took the opportunity to build the from with divs rather than a
table since we're going to need a more flexible layout when the time
comes. Also less DOM nodes.

For adding files to sample_sets. Alters the existing id strings by
removing dates.
I wanted to clean things up a little and alter the way id_strings are
created/decoded before making them a little more flexible in order to
produce more aesthetically pleasing id_strings.

this is a refactor of anon_names. We want to use anon_names elsewhere
but without the id when the user is admin. display_names replaces
anon_names and behaves as anon_names did before.

set the cursor to a pointer when hovering the cross icon inside the
token span

See #2976