[Catalyst-commits] r11670 -
Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial
tome at dev.catalyst.perl.org
tome at dev.catalyst.perl.org
Sat Oct 24 02:44:18 GMT 2009
Author: tome
Date: 2009-10-24 02:44:17 +0000 (Sat, 24 Oct 2009)
New Revision: 11670
Modified:
Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/03_MoreCatalystBasics.pod
Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/04_BasicCRUD.pod
Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/05_Authentication.pod
Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/08_Testing.pod
Log:
Fix up a few small bugs in the tutorial.
Tutorial/03_MoreCatalystBasics.pod
- Change two references from DBIx::Class:Schema::Loader to DBIx::Class::Schema::Loader
- Remove reference to components=timestamp (not in the command, only in description) since Tutorial/04 is where it is first used, and there it is described as being added to the command.
Tutorial/04_BasicCRUD.pod
- properly indent pod for author_count sub (it should be in the sub example, not as pod in the tutorial).
Tutorial/05_Authentication.pod
- The myapp.conf configuration for Plugin::Authentication has the use_session 1 entry. However, use_session is not explicit in the MyApp.pm example above this section. Since use_session is not required (Catalyst::Plugin::Session is being used) and it does not exist in the MyApp.pm which the conf is supposed to follow and it is not discussed in the tutorial text, I suggest we omit it.
Tutorial/08_Testing.pod
- Change test for 'Create' to 'Admin Create' (see bug #49825 http://rt.cpan.org/Public/Bug/Display.html?id=49825)
Modified: Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/03_MoreCatalystBasics.pod
===================================================================
--- Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/03_MoreCatalystBasics.pod 2009-10-24 02:03:28 UTC (rev 11669)
+++ Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/03_MoreCatalystBasics.pod 2009-10-24 02:44:17 UTC (rev 11670)
@@ -649,13 +649,13 @@
your OS command prompt.
Please note that here we have chosen to use 'singular' table names. This
-is because the default inflection code for L<DBIx::Class:Schema::Loader>
+is because the default inflection code for L<DBIx::Class::Schema::Loader>
does NOT handle plurals. There has been much philosophical discussion
on whether table names should be plural or singular. There is no one
correct answer, as long as one makes a choice and remains consistent
with it. If you prefer plural table names (e.g. they are easier and
more natural to read) then you will need to pass it an inflect_map
-option. See L<DBIx::Class:Schema::Loader> for more information.
+option. See L<DBIx::Class::Schema::Loader> for more information.
For using other databases, such as PostgreSQL or MySQL, see
L<Appendix 2|Catalyst::Manual::Tutorial::10_Appendices>.
@@ -754,11 +754,6 @@
=item *
-C<components=TimeStamp> causes the help to include the
-L<DBIx::Class::TimeStamp|DBIx::Class::TimeStamp> DBIC component.
-
-=item *
-
And finally, C<dbi:SQLite:myapp.db> is the standard DBI connect string
for use with SQLite.
Modified: Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/04_BasicCRUD.pod
===================================================================
--- Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/04_BasicCRUD.pod 2009-10-24 02:03:28 UTC (rev 11669)
+++ Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/04_BasicCRUD.pod 2009-10-24 02:44:17 UTC (rev 11670)
@@ -1362,9 +1362,9 @@
return the number of authors for a book. Open
C<lib/MyApp/Schema/Result/Book.pm> and add the following method:
-=head2 author_count
+ =head2 author_count
-Return the number of authors for the current book
+ Return the number of authors for the current book
=cut
Modified: Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/05_Authentication.pod
===================================================================
--- Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/05_Authentication.pod 2009-10-24 02:03:28 UTC (rev 11669)
+++ Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/05_Authentication.pod 2009-10-24 02:44:17 UTC (rev 11670)
@@ -334,7 +334,6 @@
to the following code:
<Plugin::Authentication>
- use_session 1
<default>
password_type clear
user_model DB::User
Modified: Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/08_Testing.pod
===================================================================
--- Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/08_Testing.pod 2009-10-24 02:03:28 UTC (rev 11669)
+++ Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial/08_Testing.pod 2009-10-24 02:44:17 UTC (rev 11670)
@@ -270,7 +270,7 @@
# Make sure the appropriate logout buttons are displayed
$_->content_contains("/logout\">User Logout</a>",
"Both users should have a 'User Logout'") for $ua1, $ua2;
- $ua1->content_contains("/books/form_create\">Create</a>",
+ $ua1->content_contains("/books/form_create\">Admin Create</a>",
"Only 'test01' should have a create link");
$ua1->get_ok("http://localhost/books/list", "View book list as 'test01'");
More information about the Catalyst-commits
mailing list