[Catalyst-commits] r11670 - Catalyst-Manual/5.80/branches/tome_tutorial_fixes/lib/Catalyst/Manual/Tutorial

[tome at dev.catalyst.perl.org]

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'");