hakanernam opened a new issue, #14652: URL: https://github.com/apache/grails-core/issues/14652
One to One relationship without any ownership: Setting one side in constructor results in reverse automatically set. I could reluctantly live with this if it is the officially supported behaviour, but would need to be explicitly documented so not not burn unsuspecting user, or disappear in a future release. Sample in: https://github.com/hakanernam/grails-onetoone ``` class Brother { String name; Sister favouriteSister static constraints = { favouriteSister nullable:true } String toString(){ "$id:$name --> ${favouriteSister?.name}" } } class Sister { String name; Brother favouriteBrother static constraints = { favouriteBrother nullable:true } String toString(){ "$id:$name --> ${favouriteBrother?.name}" } } ``` and BrotherController action, setting favouriteSister in the constructor: ``` @Transactional def setFavouriteSister(){ Sister sister = Sister.get(params.id) log.info("Sister before:" + sister) def brother = new Brother(favouriteSister: sister); brother.name = "Harry"; log.info("Brother: " + brother) log.info("Sister after:" + sister) brother.save(flush:true, failOnError:true) forward action:"index" } INFO grails.app.controllers.onetoone.BrotherController - Sister before:2:Mary --> null INFO grails.app.controllers.onetoone.BrotherController - Brother: null:Harry --> Mary INFO grails.app.controllers.onetoone.BrotherController - Sister after:2:Mary --> Harry ``` results in automatic setting of favouriteBrother of the Sister instance. This is not what I expected, given there is no ownership in the relationship. In my actual use case, this is okay, although made me chase through my code, so not in line with least surprise principle, so I came up with the example, I may be her favourite Brother, but the reverse is not necessarily true.   --- In the SisterController, if we set favouriteBrother explicitly, then reverse direction is not set, which I believe the correct result. ``` @Transactional def setFavouriteBrother(){ Brother bro = Brother.get(params.id) log.info("Brother before:" + bro) def sister = new Sister(); sister.name = "Jane" sister.favouriteBrother = bro; log.info("Sister: " + sister) log.info("Brother after:" + bro) sister.save(flush:true, failOnError:true) forward action:"index" } INFO grails.app.controllers.onetoone.SisterController - Brother before:1:John --> null INFO grails.app.controllers.onetoone.SisterController - Sister: null:Jane --> John INFO grails.app.controllers.onetoone.SisterController - Brother after:1:John --> null ```   -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
