search

GrailsInAction / graina2

Source code for the 2nd edition of Grails in Action
90 stars 91 forks source link

MEAP v8 ch.3 - 3.4.3 (p82) : wrong references #11

Open LukeMarlin opened 11 years ago

LukeMarlin commented 11 years ago

Current :

In listing 3.15, we’re able to call User.addToTags() because Tag belongsTo User. We’re also able to call Post.addToTags() because Tag belongsTo Post. But Post doesn’t belongTo Tag, so we can’t call Tag.addToPosts().

The only part where "User.addToTags()" is called is in listing 3.19, it's the same for Post.addToTags(). Maybe it's in the wrong place ?

Little suggestion :

But as shown in listing 3.16, Post doesn’t belongTo Tag, so we can’t call Tag.addToPosts().

Explanation :

The important part is changing the place of the paragraph, but I think it won't hurt to reference 3.16 in order to explain the last sentence.

From : http://www.manning-sandbox.com/thread.jspa?messageID=141570&#141570

pledbrook commented 11 years ago

Good catch. That should definitely be "In listing 3.19, ...". I think that sidebar could do with moving to either just before or just after listing 3.19 as well. @glenasmith, thoughts?