Create Home controller - Githubissues

AndrejLavrionovic commented 7 years ago

Workgin on it. There are couple issues with Home pojo and jsp form synchronization.

Controller is working fine. It does nothing at this point but tests showed that configurations are fine.

willhogan11 commented 7 years ago

Address has been reduced to just one line (String).

AndrejLavrionovic commented 7 years ago

Issue with form#1:

Values for checkboxes must return something significant. Was "" (empty string), changed for "true" - what we need.

AndrejLavrionovic commented 7 years ago

Issue with form#2:

Enum PropertyType must be associated with strings because mongodb doesn't understand the type 'PropertyType'. Enum PropertyType is changed to associate the values with strings.
If PropertyType enum is used to determine specifications then values that gathered from frontside must match the values that are provided by enum considering case. No values given at all. Enum contains 7 specifications, form contains just 4. Fixed.
petFriendly is a boolean type. Boolean accept only two values: true and false, nothing else. It cannot understand value: 'I don't mind.' Removed 'I don't mind' option from form.
petFriendly must return at least something, returns nothing. Added values to petFriendly select control.
userVisibility should be changed for homeVisibility.
Enum homeVisibility or userVisibility must be associated with strings because mongodb doesn't understand the type of 'UserVisibility'. Enum UserVisibility is changed to associate the values with strings.
jsp form for userVisibility >> No values given at all. All fixed.

AndrejLavrionovic commented 7 years ago

All data is associated with Home pojo. Thanks Will, brilliant work. Controller works fine. Next:

AndrejLavrionovic commented 7 years ago

Security > user must be authenticated to add home (home must be linked to user) - DONE.

addhome.jsp >> firefox is not shown calendar on available date form - working on it.

AndrejLavrionovic commented 7 years ago

addhome.jsp >> firefox is not shown calendar on available date form - working on it. - FIXED. Doing testing.

AndrejLavrionovic commented 7 years ago

addhome is done except images uploading

willhogan11 commented 7 years ago

Cool, what was the issue with Firefox? I though the 'Webshims' javascript code would have fixed it like with the 'apply page'?

willhogan11 commented 7 years ago

Can this be closed now?

willhogan11 / Golf_n_HomeSwap