I talked to SineSwiper on IRC and readded the link because the other
<br>
$result occurrences also have the link.
<br>
He also explained the purpose of Manual::ResultClass to me and we agreed
<br>
that splitting it into a setup section, that holds all the links to classes
<br>
like ResultSource that are needed when setting up DBIC, which are mostly,
<br>
if no all, class methods, and a use-time section with links to classes like
<br>
PK and Row.
<br>
<br>
<br>
On Fri, May 24, 2013 at 11:50 AM, dbsrgits sync <notifications@github.com>wrote:
<br>
<br>
> On Fri, May 24, 2013 at 02:30:39AM -0700, Alexander Hartmaier wrote:
<br>
> > ... Row.pm which documents the methods a user might be interested in ...
<br>
>
<br>
> Yes - the document that lists these methods is ::Manual::ResultClass,
<br>
> *not* ::Row.
<br>
>
<br>
>
<br>
> > Also mst said that update returning anything is a mistake that's hard
<br>
> > to fix now.
<br>
>
<br>
> I bet this isn't actually what he said, as he has always been a strong
<br>
> advocate of call chaining. You probably got confused by his belief
<br>
> update() *itself* is a mistake, about which I agreed to viciously
<br>
> disagree with him ;)
<br>
>
<br>
> In any case - this is a documentation decision I defer to SineSwiper,
<br>
> which has a better eye for documentation than me and you (and definitely
<br>
> better than mst ;) Please ask him on IRC or per email, preferrably
<br>
> before merging this particular line-change (the rest is fine).
<br>
>
<br>
> Cheers
<br>
>
<br>
> —
<br>
> Reply to this email directly or view it on GitHub<https://github.com/dbsrgits/dbix-class/commit/878562e696c36755dd36b83292eab878c92ef81a#commitcomment-3282068>
<br>
> .
<br>
>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/dbsrgits/dbix-class/commit/878562e696c36755dd36b83292eab878c92ef81a#commitcomment-3335607'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/4W3BWTo7EIi_7vx28Xzl_k90NEJFcIvl5wEsgoFBJcs1_nUVhKteKq-6QNR5tFZQ.gif' height='1' width='1'></p>