search

fideloper / Implementing-Laravel

Companion application to the e-book Implementing Laravel
171 stars 40 forks source link

Possible typos #5

Closed fideloper closed 11 years ago

fideloper commented 11 years ago

From an email received:

Pg. 37: Line 93: 'articles' should be '$articles' Pg. 39: Line 23: 'byPage($page, $rpp)' should be 'byPage($page, $perPage)' Pg. 56: Why do you switch from 'contentController' to 'articleController'? Pg. 61: Line 3: 'class ArticleFormValidator extends LaravelValidatorAbstract' should be 'class ArticleFormValidator extends AbstractLaravelValidator' Pg. 62: Line 3: should be 'use Impl\Service\Validation\ArticleFormValidator' right? You are referring to a class that is not created until page 66. Pg. 74: Line 9: Should there also be a 'protected $article;' here?

Lechus commented 11 years ago

Page 10(pdf): ...interfaces, Doing so should be...

Lechus commented 11 years ago

Page number 21 - There is no info that reader need linux, and you do not show alternative for Windows.

ericmartel commented 11 years ago

Page 53: 2. Invalidate the “article” cache by changing the the “articles” namespace in our ServiceProvider double "the"

fideloper commented 11 years ago

Updated for next release, thanks.