#91 - Add optional field 'links' to Thesis

[pehala]

Overview

First change where I changed Domain Model. I created a new hasMany relationship in Thesis with newly created Domain class Link. After that I updated ThesisController and ThesisService to process links. Finally I did some changes to views to reflect the changes.

Changes in files

BootStrap.groovy

• Added some of links to development enviroment

Link.groovy

• New Domain class, with just one field: url

Thesis.groovy

• Added hasMany relationship with Link

ThesisController.groovy

• Updated update() method to process links.
  • I had to bypass bindData(), because I wasn't able run it with it

ThesisService.groovy

• Added methods for processing String and String[] into Set of Links, because form return either String or String[] depending on number of links

thesis/_form.bsp

• Added Add links section
  • based on email domains in site configuration

thesis/show.bsp

• Now showing links
  • based on supervisors part

[pehala]

• id is now generated sequence instead of being composite of url.
• transformLinks() now returns List instead of Set.
• _form.bsp now process existing links as List instead of Set.

[VaclavDedik]

This looks good, two more things though:

• The documentation still says "Set with one Link" and "Set with Links", instead of List.
• You need to create a database migration for this change, otherwise I won't be able to deploy this to production. The database migration will need to create a new table Link with the necessary relationships.

[pehala]

• Documentation fixed
• Database migration generated, worked on my localhost postgre with grails dbm_update
• Added declaration to links in Thesis.groovy to make sure links are always List and not Set

[VaclavDedik]

Merged, thanks, but there is a bug, see #114

(noticed it only after I pushed the changes to diplomky.redhat.com)

[VaclavDedik]

I fixed, the issue, was trivial