[Rt-commit] r13085 - in rt/branches/3.8-TESTING: . bin etc lib/RT/I18N lib/RT/Interface sbin share/html/Elements share/html/Install share/html/Prefs share/html/REST/1.0 share/html/REST/1.0/search share/html/Search/Elements share/html/Ticket/Elements share/html/User share/html/Widgets

jesse at bestpractical.com jesse at bestpractical.com
Mon Jun 9 17:24:06 EDT 2008


Author: jesse
Date: Mon Jun  9 17:24:04 2008
New Revision: 13085

Modified:
   rt/branches/3.8-TESTING/   (props changed)
   rt/branches/3.8-TESTING/bin/rt.in
   rt/branches/3.8-TESTING/etc/RT_Config.pm.in
   rt/branches/3.8-TESTING/lib/RT/Config.pm
   rt/branches/3.8-TESTING/lib/RT/I18N/fr.po
   rt/branches/3.8-TESTING/lib/RT/Interface/REST.pm
   rt/branches/3.8-TESTING/lib/RT/Test.pm
   rt/branches/3.8-TESTING/sbin/extract-message-catalog
   rt/branches/3.8-TESTING/share/html/Elements/CollectionList
   rt/branches/3.8-TESTING/share/html/Elements/CollectionListPaging
   rt/branches/3.8-TESTING/share/html/Elements/TicketList
   rt/branches/3.8-TESTING/share/html/Install/Emails.html
   rt/branches/3.8-TESTING/share/html/Install/Sendmail.html
   rt/branches/3.8-TESTING/share/html/Prefs/Other.html
   rt/branches/3.8-TESTING/share/html/REST/1.0/dhandler
   rt/branches/3.8-TESTING/share/html/REST/1.0/search/ticket
   rt/branches/3.8-TESTING/share/html/Search/Elements/PickCFs
   rt/branches/3.8-TESTING/share/html/Ticket/Elements/UpdateCc
   rt/branches/3.8-TESTING/share/html/User/Prefs.html
   rt/branches/3.8-TESTING/share/html/Widgets/TitleBoxStart

Log:


Modified: rt/branches/3.8-TESTING/bin/rt.in
==============================================================================
--- rt/branches/3.8-TESTING/bin/rt.in	(original)
+++ rt/branches/3.8-TESTING/bin/rt.in	Mon Jun  9 17:24:04 2008
@@ -94,7 +94,7 @@
 # (XXX: Ask Autrijus how i18n changes these definitions.)
 
 my $name   = '[\w.-]+';
-my $field  = '(?:[a-zA-Z](?:[a-zA-Z0-9_-]|\s+)*)';
+my $field  = '(?i:[a-z][a-z0-9_-]*|C(?:ustom)?F(?:ield)?-(?:[a-z0-9_ :()/-]|\s)+)';
 my $label  = '[a-zA-Z0-9 at _.+-]+';
 my $labels = "(?:$label,)*$label";
 my $idlist = '(?:(?:\d+-)?\d+,)*(?:\d+-)?\d+';
@@ -455,7 +455,7 @@
             whine "What type of object do you want to create?";
             $bad = 1;
         }
-        @objects = ("$type/new");
+        @objects = ("$type/new") if defined($type);
     }
     #return help($action, $type) if $bad;
     return suggest_help($action, $type) if $bad;

Modified: rt/branches/3.8-TESTING/etc/RT_Config.pm.in
==============================================================================
--- rt/branches/3.8-TESTING/etc/RT_Config.pm.in	(original)
+++ rt/branches/3.8-TESTING/etc/RT_Config.pm.in	Mon Jun  9 17:24:04 2008
@@ -1,7 +1,3 @@
-#
-# WARNING: NEVER EDIT RT_Config.pm. Instead, copy any sections you want to change to RT_SiteConfig.pm
-# and edit them there.
-#
 
 package RT;
 
@@ -15,327 +11,579 @@
 
 =cut
 
-# {{{ Base Configuration
+=head1 WARNING
 
-# $rtname is the string that RT will look for in mail messages to
-# figure out what ticket a new piece of mail belongs to
+NEVER EDIT RT_Config.pm.
 
-# Your domain name is recommended, so as not to pollute the namespace.
-# once you start using a given tag, you should probably never change it.
-# (otherwise, mail for existing tickets won't get put in the right place
+Instead, copy any sections you want to change to F<RT_SiteConfig.pm> and edit them there.
+
+=cut
+
+=head1 Base Configuration
+
+=over 4
+
+=item C<$rtname>
+
+C<$rtname> is the string that RT will look for in mail messages to
+figure out what ticket a new piece of mail belongs to.
+
+Your domain name is recommended, so as not to pollute the namespace.
+once you start using a given tag, you should probably never change it.
+(otherwise, mail for existing tickets won't get put in the right place)
+
+=cut
 
 Set($rtname , "example.com");
 
 
-# This regexp controls what subject tags RT recognizes as its own.
-# If you're not dealing with historical $rtname values, you'll likely
-# never have to enable this feature.
-#
-# Be VERY CAREFUL with it. Note that it overrides $rtname for subject
-# token matching and that you should use only "non-capturing" parenthesis
-# grouping. For example:
-#
-#     Set($EmailSubjectTagRegex, qr/(?:example.com|example.org)/i );
-#
-# and NOT
-# 
-#     Set($EmailSubjectTagRegex, qr/(example.com|example.org)/i );
-#
-# This setting would make RT behave exactly as it does without the 
-# setting enabled.
-#
-# Set($EmailSubjectTagRegex, qr/\Q$rtname\E/i );
+=item C<$EmailSubjectTagRegex>
+
+This regexp controls what subject tags RT recognizes as its own.
+If you're not dealing with historical C<$rtname> values, you'll likely
+never have to enable this feature.
+
+Be VERY CAREFUL with it. Note that it overrides C<$rtname> for subject
+token matching and that you should use only "non-capturing" parenthesis
+grouping. For example:
+
+C<Set($EmailSubjectTagRegex, qr/(?:example.com|example.org)/i );>
 
+and NOT
+
+C<Set($EmailSubjectTagRegex, qr/(example.com|example.org)/i );>
+
+This setting would make RT behave exactly as it does without the 
+setting enabled.
+
+=cut
 
+#Set($EmailSubjectTagRegex, qr/\Q$rtname\E/i );
 
-# You should set this to your organization's DNS domain. For example,
-# fsck.com or asylum.arkham.ma.us. It's used by the linking interface to
-# guarantee that ticket URIs are unique and easy to construct.
+
+
+=item C<$Organization>
+
+You should set this to your organization's DNS domain. For example,
+I<fsck.com> or I<asylum.arkham.ma.us>. It's used by the linking interface to
+guarantee that ticket URIs are unique and easy to construct.
+
+=cut
 
 Set($Organization , "example.com");
 
-# $MinimumPasswordLength defines the minimum length for user
-# passwords. Setting it to 0 disables this check
+=item C<$MinimumPasswordLength>
+
+C<$MinimumPasswordLength> defines the minimum length for user
+passwords. Setting it to 0 disables this check.
+
+=cut
+
 Set($MinimumPasswordLength , "5");
 
-# $Timezone is used to convert times entered by users into GMT and back again
-# It should be set to a timezone recognized by your local unix box.
+=item C<$Timezone>
+
+C<$Timezone> is used to convert times entered by users into GMT and back again
+It should be set to a timezone recognized by your local unix box.
+
+=cut
+
 Set($Timezone , 'US/Eastern');
 
-# }}}
+=back
+
+=head1 Database Configuration
+
+=over 4
 
-# {{{ Database Configuration
+=item C<$DatabaseType>
 
-# Database driver being used; case matters.  Valid types are "mysql",
-# "Oracle" and "Pg"
+Database driver being used; case matters.
+
+Valid types are "mysql", "Oracle" and "Pg"
+
+=cut
 
 Set($DatabaseType , '@DB_TYPE@');
 
-# The domain name of your database server
-# If you're running mysql and it's on localhost,
-# leave it blank for enhanced performance
+=item C<$DatabaseHost>, C<$DatabaseRTHost>
+
+The domain name of your database server.
+
+If you're running mysql and it's on localhost,
+leave it blank for enhanced performance
+
+=cut
+
 Set($DatabaseHost   , '@DB_HOST@');
 Set($DatabaseRTHost , '@DB_RT_HOST@');
 
-# The port that your database server is running on.  Ignored unless it's
-# a positive integer. It's usually safe to leave this blank
+=item C<$DatabasePort>
+
+The port that your database server is running on.  Ignored unless it's
+a positive integer. It's usually safe to leave this blank
+
+=cut
+
 Set($DatabasePort , '@DB_PORT@');
 
-#The name of the database user (inside the database)
+=item C<$DatabaseUser>
+
+The name of the database user (inside the database)
+
+=cut
+
 Set($DatabaseUser , '@DB_RT_USER@');
 
-# Password the DatabaseUser should use to access the database
+=item C<$DatabasePassword>
+
+Password the C<$DatabaseUser> should use to access the database
+
+=cut
+
 Set($DatabasePassword , '@DB_RT_PASS@');
 
-# The name of the RT's database on your database server
+=item C<$DatabaseName>
+
+The name of the RT's database on your database server
+
+=cut
+
 Set($DatabaseName , '@DB_DATABASE@');
 
-# If you're using Postgres and have compiled in SSL support,
-# set DatabaseRequireSSL to 1 to turn on SSL communication
+=item C<$DatabaseRequireSSL>
+
+If you're using Postgres and have compiled in SSL support,
+set C<$DatabaseRequireSSL> to 1 to turn on SSL communication
+
+=cut
+
 Set($DatabaseRequireSSL , undef);
 
-# }}}
+=back
+
+=head1 Incoming Mail Gateway Configuration
+
+=over 4
 
-# {{{ Incoming mail gateway configuration
+=item C<$OwnerEmail>
 
-# OwnerEmail is the address of a human who manages RT. RT will send
-# errors generated by the mail gateway to this address.  This address
-# should _not_ be an address that's managed by your RT instance.
+C<$OwnerEmail> is the address of a human who manages RT. RT will send
+errors generated by the mail gateway to this address.  This address
+should _not_ be an address that's managed by your RT instance.
+
+=cut
 
 Set($OwnerEmail , 'root');
 
-# If $LoopsToRTOwner is defined, RT will send mail that it believes
-# might be a loop to $OwnerEmail
+=item C<$LoopsToRTOwner>
+
+If C<$LoopsToRTOwner> is defined, RT will send mail that it believes
+might be a loop to C<$OwnerEmail>
+
+=cut
 
 Set($LoopsToRTOwner , 1);
 
-# If $StoreLoops is defined, RT will record messages that it believes
-# to be part of mail loops.
-# As it does this, it will try to be careful not to send mail to the
-# sender of these messages
+=item C<$StoreLoops>
+
+If C<$StoreLoops> is defined, RT will record messages that it believes
+to be part of mail loops.
+
+As it does this, it will try to be careful not to send mail to the
+sender of these messages
+
+=cut
 
 Set($StoreLoops , undef);
 
-# $MaxAttachmentSize sets the maximum size (in bytes) of attachments stored
-# in the database.
+=item C<$MaxAttachmentSize>
+
+C<$MaxAttachmentSize> sets the maximum size (in bytes) of attachments stored
+in the database.
+
+For mysql and oracle, we set this size at 10 megabytes.
+If you're running a postgres version earlier than 7.1, you will need
+to drop this to 8192. (8k)
+
+=cut
 
-# For mysql and oracle, we set this size at 10 megabytes.
-# If you're running a postgres version earlier than 7.1, you will need
-# to drop this to 8192. (8k)
 
 Set($MaxAttachmentSize , 10000000);
 
-# $TruncateLongAttachments: if this is set to a non-undef value,
-# RT will truncate attachments longer than MaxAttachmentSize.
+=item C<$TruncateLongAttachments>
+
+C<$TruncateLongAttachments>: if this is set to a non-undef value,
+RT will truncate attachments longer than C<$MaxAttachmentSize>.
+
+=cut
 
 Set($TruncateLongAttachments , undef);
 
-# $DropLongAttachments: if this is set to a non-undef value,
-# RT will silently drop attachments longer than MaxAttachmentSize.
+=item C<$DropLongAttachments>
+
+C<$DropLongAttachments>: if this is set to a non-undef value,
+RT will silently drop attachments longer than C<MaxAttachmentSize>.
+
+=cut
 
 Set($DropLongAttachments , undef);
 
-# If $ParseNewMessageForTicketCcs is true, RT will attempt to divine
-# Ticket 'Cc' watchers from the To and Cc lines of incoming messages
-# Be forewarned that if you have _any_ addresses which forward mail to
-# RT automatically and you enable this option without modifying
-# "RTAddressRegexp" below, you will get yourself into a heap of trouble.
+=item C<$ParsenewMessageForTicketCcs>
+
+If C<$ParseNewMessageForTicketCcs> is true, RT will attempt to divine
+Ticket 'Cc' watchers from the To and Cc lines of incoming messages
+Be forewarned that if you have _any_ addresses which forward mail to
+RT automatically and you enable this option without modifying
+C<$RTAddressRegexp> below, you will get yourself into a heap of trouble.
+
+=cut
 
 Set($ParseNewMessageForTicketCcs , undef);
 
-# RTAddressRegexp is used to make sure RT doesn't add itself as a ticket CC if
-# the setting above is enabled.
+=item C<$RTAddressRegexp> 
+
+C<$RTAddressRegexp> is used to make sure RT doesn't add itself as a ticket CC if
+the setting above is enabled.
+
+=cut
 
 Set($RTAddressRegexp , '^rt\@example.com$');
 
-# RT provides functionality which allows the system to rewrite
-# incoming email addresses.  In its simplest form,
-# you can substitute the value in CanonicalizeEmailAddressReplace
-# for the value in CanonicalizeEmailAddressMatch
-# (These values are passed to the CanonicalizeEmailAddress subroutine in RT/User.pm)
-# By default, that routine performs a s/$Match/$Replace/gi on any address passed to it
+=item C<$CanonicalizeEmailAddressMatch>, C<$CanonicalizeEmailAddressReplace>
+
+RT provides functionality which allows the system to rewrite
+incoming email addresses.  In its simplest form,
+you can substitute the value in $<CanonicalizeEmailAddressReplace>
+for the value in $<CanonicalizeEmailAddressMatch>
+(These values are passed to the $<CanonicalizeEmailAddress> subroutine in
+ F<RT/User.pm>)
+
+By default, that routine performs a C<s/$Match/$Replace/gi> on any address
+passed to it.
+
+=cut
 
 #Set($CanonicalizeEmailAddressMatch , '@subdomain\.example\.com$');
 #Set($CanonicalizeEmailAddressReplace , '@example.com');
 
-# set this to true and the create new user page will use the values that you
-# enter in the form but use the function CanonicalizeUserInfo in User_Local.pm
-Set($CanonicalizeOnCreate , 0);
-
-# If $SenderMustExistInExternalDatabase is true, RT will refuse to
-# create non-privileged accounts for unknown users if you are using
-# the "LookupSenderInExternalDatabase" option.
-# Instead, an error message will be mailed and RT will forward the
-# message to $RTOwner.
-#
-# If you are not using $LookupSenderInExternalDatabase, this option
-# has no effect.
-#
-# If you define an AutoRejectRequest template, RT will use this
-# template for the rejection message.
+=item C<$CanonicalizeEmailAddressMatch>
+
+Set this to true and the create new user page will use the values that you
+enter in the form but use the function CanonicalizeUserInfo in
+F<RT/User_Local.pm>
+
+=cut
+
+Set($CanonicalizeOnCreate, 0);
+
+=item C<$SenderMustExistInExternalDatabase>
+
+If C<$SenderMustExistInExternalDatabase> is true, RT will refuse to
+create non-privileged accounts for unknown users if you are using
+the C<$LookupSenderInExternalDatabase> option.
+Instead, an error message will be mailed and RT will forward the
+message to C<$RTOwner>.
+
+If you are not using C<$LookupSenderInExternalDatabase>, this option
+has no effect.
+
+If you define an AutoRejectRequest template, RT will use this
+template for the rejection message.
+
+=cut
 
 Set($SenderMustExistInExternalDatabase , undef);
 
-# @MailPlugins is a list of auth plugins for L<RT::Interface::Email>
-# to use; see L<rt-mailgate>
+=item C<@MailPlugins>
+
+C<@MailPlugins> is a list of auth plugins for L<RT::Interface::Email>
+to use; see L<rt-mailgate>
+
+=cut
+
+=item C<$UnsafeEmailCommands>
+
+C<$UnsafeEmailCommands>, if set to true, enables 'take' and 'resolve'
+as possible actions via the mail gateway.  As its name implies, this
+is very unsafe, as it allows email with a forged sender to possibly
+resolve arbitrary tickets!
+
+=cut
+
+=item C<$ExtractSubjectTagMatch>, C<$ExtractSubjectTagNoMatch>
 
-# $UnsafeEmailCommands, if set to true, enables 'take' and 'resolve'
-# as possible actions via the mail gateway.  As its name implies, this
-# is very unsafe, as it allows email with a forged sender to possibly
-# resolve arbitrary tickets!
+The default "extract remote tracking tags" scrip settings; these
+detect when your RT is talking to another RT, and adjusts the
+subject accordingly.
 
+=cut
 
-# The default "extract remote tracking tags" scrip settings; these
-# detect when your RT is talking to another RT, and adjusts the
-# subject accordingly.
 Set($ExtractSubjectTagMatch, qr/\[.+? #\d+\]/);
 Set($ExtractSubjectTagNoMatch, ( ${RT::EmailSubjectTagRegex}
        ? qr/\[(?:${RT::EmailSubjectTagRegex}) #\d+\]/
        : qr/\[\Q$RT::rtname\E #\d+\]/));
 
-# }}}
+=back
+
+=head1 Outgoing Mail Configuration
 
-# {{{ Outgoing mail configuration
+=over 4
 
-# $MailCommand defines which method RT will use to try to send mail.
-# We know that 'sendmailpipe' works fairly well.  If 'sendmailpipe'
-# doesn't work well for you, try 'sendmail'.  Other options are 'smtp'
-# or 'qmail'.
-#
-# Note that you should remove the '-t' from $SendmailArguments
-# if you use 'sendmail' rather than 'sendmailpipe'
+=item C<$MailCommand>
+
+C<$MailCommand> defines which method RT will use to try to send mail.
+We know that 'sendmailpipe' works fairly well.  If 'sendmailpipe'
+doesn't work well for you, try 'sendmail'.  Other options are 'smtp'
+or 'qmail'.
+
+Note that you should remove the '-t' from C<$SendmailArguments>
+if you use 'sendmail' rather than 'sendmailpipe'
+
+=cut
 
 Set($MailCommand , 'sendmailpipe');
 
-# }}}
+=back
+
+=head1 Sendmail Configuration
+
+These options only take effect if C<$MailCommand> is 'sendmail' or
+'sendmailpipe'
+
+=over 4
+
+=item C<$SendmailArguments> 
+
+C<$SendmailArguments> defines what flags to pass to C<$SendmailPath>
+If you picked 'sendmailpipe', you MUST add a -t flag to C<$SendmailArguments>
+These options are good for most sendmail wrappers and workalikes
+
+These arguments are good for sendmail brand sendmail 8 and newer
+C<Set($SendmailArguments,"-oi -t -ODeliveryMode=b -OErrorMode=m");>
+
+=cut
 
-# {{{ Sendmail Configuration
-# These options only take effect if $MailCommand is 'sendmail' or
-# 'sendmailpipe'
-
-# $SendmailArguments defines what flags to pass to $SendmailPath
-# If you picked 'sendmailpipe', you MUST add a -t flag to $SendmailArguments
-# These options are good for most sendmail wrappers and workalikes
 Set($SendmailArguments , "-oi -t");
-# These arguments are good for sendmail brand sendmail 8 and newer
-#Set($SendmailArguments,"-oi -t -ODeliveryMode=b -OErrorMode=m");
 
-# $SendmailBounceArguments defines what flags to pass to $Sendmail
-# assuming RT needs to send an error (ie. bounce).
+
+=item C<$SendmailBounceArguments>
+
+C<$SendmailBounceArguments> defines what flags to pass to C<$Sendmail>
+assuming RT needs to send an error (ie. bounce).
+
+=cut
+
 Set($SendmailBounceArguments , '-f "<>"');
-# If you selected 'sendmailpipe' above, you MUST specify the path to
-# your sendmail binary in $SendmailPath.
+
+=item C<$SendmailPath>
+
+If you selected 'sendmailpipe' above, you MUST specify the path to
+your sendmail binary in C<$SendmailPath>.
+
+=cut
+
 Set($SendmailPath , "/usr/sbin/sendmail");
 
-# }}}
 
-# {{{ SMTP configuration
-# These options only take effect if $MailCommand is 'smtp'
+=back
+
+=head1 SMTP Configuration
+
+These options only take effect if C<$MailCommand> is 'smtp'
+
+=over 4
+
+=item C<$SMTPServer>
+
+C<$SMTPServer> should be set to the hostname of the SMTP server to use
+
+=cut
 
-# $SMTPServer should be set to the hostname of the SMTP server to use
 Set($SMTPServer, undef);
-# $SMTPFrom should be set to the 'From' address to use, if not the
-# email's 'From'
+
+=item C<$SMTPFrom>
+
+C<$SMTPFrom> should be set to the 'From' address to use, if not the
+email's 'From'
+
+=cut
+
 Set($SMTPFrom, undef);
-# $SMTPDebug should be set to true to debug SMTP mail sending
+
+=item C<$SMTPDebug> 
+
+C<$SMTPDebug> should be set to true to debug SMTP mail sending
+
+=cut
+
 Set($SMTPDebug, 0);
-# }}}
 
-# {{{ Other mailer configuration
-# @MailParams defines a list of options passed to $MailCommand if it
-# is not 'sendmailpipe', 'sendmail', or 'smtp'
+=back
+
+=head1 Other Mailer Configuration
+
+=over 4
+
+=item C<@MailParams>
+
+C<@MailParams> defines a list of options passed to $MailCommand if it
+is not 'sendmailpipe', 'sendmail', or 'smtp'
+
+=cut
+
 Set(@MailParams, ());
 
-# RT is designed such that any mail which already has a ticket-id associated
-# with it will get to the right place automatically.
+=item C<$CorrespondAddress>, C<$CommentAddress>
+
+RT is designed such that any mail which already has a ticket-id associated
+with it will get to the right place automatically.
 
-# $CorrespondAddress and $CommentAddress are the default addresses
-# that will be listed in From: and Reply-To: headers of correspondence
-# and comment mail tracked by RT, unless overridden by a queue-specific
-# address.
+C<$CorrespondAddress> and C<$CommentAddress> are the default addresses
+that will be listed in From: and Reply-To: headers of correspondence
+and comment mail tracked by RT, unless overridden by a queue-specific
+address.
+
+=cut
 
 Set($CorrespondAddress , '');
 
 Set($CommentAddress , '');
 
-# By default, RT sets the outgoing mail's "From:" header to
-# "SenderName via RT".  Setting UseFriendlyFromLine to 0 disables it.
+=item C<UseFriendlyFromLine>
+
+By default, RT sets the outgoing mail's "From:" header to
+"SenderName via RT".  Setting C<$UseFriendlyFromLine> to 0 disables it.
+
+=cut
+
+Set($UseFriendlyFromLine, 1);
+
+=item C<$FriendlyFromLineFormat>
+
+C<sprintf()> format of the friendly 'From:' header; its arguments
+are SenderName and SenderEmailAddress.
+
+=cut
+
+Set($FriendlyFromLineFormat, "\"%s via RT\" <%s>");
+
+=item C<$UseFriendlyToLine>
+
+RT can optionally set a "Friendly" 'To:' header when sending messages to
+Ccs or AdminCcs (rather than having a blank 'To:' header.
 
-Set($UseFriendlyFromLine , 1);
+This feature DOES NOT WORK WITH SENDMAIL[tm] BRAND SENDMAIL
+If you are using sendmail, rather than postfix, qmail, exim or some other MTA,
+you _must_ disable this option.
 
-# sprintf() format of the friendly 'From:' header; its arguments
-# are SenderName and SenderEmailAddress.
-Set($FriendlyFromLineFormat , "\"%s via RT\" <%s>");
+=cut
+
+Set($UseFriendlyToLine, 0);
 
-# RT can optionally set a "Friendly" 'To:' header when sending messages to
-# Ccs or AdminCcs (rather than having a blank 'To:' header.
+=item C<$FriendlyToLineFormat>
 
-# This feature DOES NOT WORK WITH SENDMAIL[tm] BRAND SENDMAIL
-# If you are using sendmail, rather than postfix, qmail, exim or some other MTA,
-# you _must_ disable this option.
+C<sprintf()> format of the friendly 'From:' header; its arguments
+are WatcherType and TicketId.
 
-Set($UseFriendlyToLine , 0);
+=cut
 
-# sprintf() format of the friendly 'From:' header; its arguments
-# are WatcherType and TicketId.
 Set($FriendlyToLineFormat, "\"%s of ". RT->Config->Get('rtname') ." Ticket #%s\":;");
 
-# By default, RT doesn't notify the person who performs an update, as they
-# already know what they've done. If you'd like to change this behaviour,
-# Set $NotifyActor to 1
+=item C<$NotifyActor>
+
+By default, RT doesn't notify the person who performs an update, as they
+already know what they've done. If you'd like to change this behaviour,
+Set C<$NotifyActor> to 1
+
+=cut
 
 Set($NotifyActor, 0);
 
-# By default, RT records each message it sends out to its own internal database.
-# To change this behavior, set $RecordOutgoingEmail to 0 
+=item C<$RecordOutgoingEmail>
+
+By default, RT records each message it sends out to its own internal database.
+To change this behavior, set C<$RecordOutgoingEmail> to 0 
+
+=cut
 
 Set($RecordOutgoingEmail, 1);
 
-# VERP support (http://cr.yp.to/proto/verp.txt)
-# uncomment the following two directives to generate envelope senders
-# of the form ${VERPPrefix}${originaladdress}@${VERPDomain}
-# (i.e. rt-jesse=fsck.com at rt.example.com ) This currently only works
-# with sendmail and sendmailppie.
+=item C<$VERPPrefix>, C<$VERPPrefix>
+
+VERP support (http://cr.yp.to/proto/verp.txt)
+
+uncomment the following two directives to generate envelope senders
+of the form C<${VERPPrefix}${originaladdress}@${VERPDomain}>
+(i.e. rt-jesse=fsck.com at rt.example.com ).
+
+This currently only works with sendmail and sendmailppie.
+
+=cut
+
 # Set($VERPPrefix, 'rt-');
 # Set($VERPDomain, $RT::Organization);
 
 
-# By default, RT forwards a message using queue's address and adds RT's tag into
-# subject of the outgoing message, so recipients' replies go into RT as correspondents.
-# To change this behavior, set $ForwardFromUser to true value and RT will use
-# address of the current user and leave subject without RT's tag.
+=item C<$ForwardFromUser>
+
+By default, RT forwards a message using queue's address and adds RT's tag into
+subject of the outgoing message, so recipients' replies go into RT as correspondents.
+
+To change this behavior, set C<$ForwardFromUser> to true value and RT will use
+address of the current user and leave subject without RT's tag.
+
+=cut
 
 Set($ForwardFromUser, 0);
 
-# By default RT hides from the web UI information about blind copies user sent on
-# reply or comment, to change this set the following option to true value
+=item C<$ShowBccHeader>
+
+By default RT hides from the web UI information about blind copies user sent on
+reply or comment.
+
+To change this set the following option to true value.
+
+=cut
+
 Set($ShowBccHeader, 0);
 
-# }}}
+=back
+
+=head1 GnuPG Configuration
+
+A full description of the (somewhat extensive) GnuPG integration can be found 
+by running the command `perldoc L<RT::Crypt::GnuPG>`  (or `perldoc
+        lib/RT/Crypt/GnuPG.pm` from your RT install directory).
 
-# {{{ GnuPG
-# A full description of the (somewhat extensive) GnuPG integration can be found by 
-# running the command 
-#
-#  perldoc RT::Crypt::GnuPG  (or perldoc lib/RT/Crypt/GnuPG.pm from your RT install directory).
+=over 4
 
+=item C<%GnuPG>
+
+Set C<OutgoingMessagesFormat> to 'inline' to use inline encryption and
+signatures instead of 'RFC' (GPG/MIME: RFC3156 and RFC1847) format.
+
+If you want to allow people to encrypt attachments inside the DB then
+set C<AllowEncryptDataInDB> to true
+
+=cut
 
 Set( %GnuPG,
     Enable => @RT_GPG@,
-
-# Set OutgoingMessagesFormat to 'inline' to use inline encryption and
-# signatures instead of 'RFC' (GPG/MIME: RFC3156 and RFC1847) format.
     OutgoingMessagesFormat => 'RFC', # Inline
-
-# If you want to allow people to encrypt attachments inside the DB then
-# set below option to true value
     AllowEncryptDataInDB   => 0,
 );
 
-# Options of GnuPG program
-# If you override this in your RT_SiteConfig, you should be sure
-# to include a homedir setting
-# NOTE that options with '-' character MUST be quoted.
+=item C<%GnuPGOptions>
+
+Options of GnuPG program.
+
+If you override this in your RT_SiteConfig, you should be sure
+to include a homedir setting.
+
+NOTE that options with '-' character MUST be quoted.
+
+=cut
 
 Set(%GnuPGOptions,
     homedir => '@RT_VAR_PATH@/data/gpg',
@@ -350,284 +598,426 @@
 #    'auto-key-retrieve' => undef,
 );
 
-# }}}
 
-# {{{ Logging
+=back
+
+=head1 Logging Configuration
+
+The default is to log anything except debugging
+information to syslog.  Check the L<Log::Dispatch> POD for
+information about how to get things by syslog, mail or anything
+else, get debugging info in the log, etc.
+
+It might generally make sense to send error and higher by email to
+some administrator.  If you do this, be careful that this email
+isn't sent to this RT instance.  Mail loops will generate a critical
+log message.
+
+=over 4
+
+=item C<$LogToSyslog>, C<$LogToScreen>
+
+The minimum level error that will be logged to the specific device.
+From lowest to highest priority, the levels are:
+ debug info notice warning error critical alert emergency
+
+=cut
 
-# Logging.  The default is to log anything except debugging
-# information to syslog.  Check the Log::Dispatch POD for
-# information about how to get things by syslog, mail or anything
-# else, get debugging info in the log, etc.
-
-# It might generally make sense to send error and higher by email to
-# some administrator.  If you do this, be careful that this email
-# isn't sent to this RT instance.  Mail loops will generate a critical
-# log message.
-
-# The minimum level error that will be logged to the specific device.
-# From lowest to highest priority, the levels are:
-#  debug info notice warning error critical alert emergency
 Set($LogToSyslog    , 'debug');
 Set($LogToScreen    , 'info');
 
-# Logging to a standalone file is also possible, but note that the
-# file should needs to both exist and be writable by all direct users
-# of the RT API.  This generally include the web server, whoever
-# rt-crontool runs as.  Note that as rt-mailgate and the RT CLI go
-# through the webserver, so their users do not need to have write
-# permissions to this file. If you expect to have multiple users of
-# the direct API, Best Practical recommends using syslog instead of
-# direct file logging.
+=item C<$LogToFile>, C<$LogDir>, C<$LogToFileNamed>
+
+Logging to a standalone file is also possible, but note that the
+file should needs to both exist and be writable by all direct users
+of the RT API.  This generally include the web server, whoever
+rt-crontool runs as.  Note that as rt-mailgate and the RT CLI go
+through the webserver, so their users do not need to have write
+permissions to this file. If you expect to have multiple users of
+the direct API, Best Practical recommends using syslog instead of
+direct file logging.
+
+=cut
 
 Set($LogToFile      , undef);
 Set($LogDir, '@RT_LOG_PATH@');
 Set($LogToFileNamed , "rt.log");    #log to rt.log
 
-# If set to a log level then logging will include stack
-# traces for messages with level equal to or greater than
-# specified.
-
-Set($LogStackTraces , '');
-
-# On Solaris or UnixWare, set to ( socket => 'inet' ).  Options here
-# override any other options RT passes to Log::Dispatch::Syslog.
-# Other interesting flags include facility and logopt.  (See the
-# Log::Dispatch::Syslog documentation for more information.)  (Maybe
-# ident too, if you have multiple RT installations.)
+=item C<$LogStackTraces>
+
+If set to a log level then logging will include stack
+traces for messages with level equal to or greater than
+specified.
+
+=cut
+
+Set($LogStackTraces, '');
+
+=item C<@LogToSyslogConf>
+
+On Solaris or UnixWare, set to ( socket => 'inet' ).  Options here
+override any other options RT passes to L<Log::Dispatch::Syslog>.
+Other interesting flags include facility and logopt.  (See the
+L<Log::Dispatch::Syslog> documentation for more information.)  (Maybe
+ident too, if you have multiple RT installations.)
+
+=cut
 
 Set(@LogToSyslogConf, ());
 
-# RT has rudimentary SQL statement logging support if you have
-# DBIx-SearchBuilder 1.31_1 or higher; simply set $StatementLog to be
-# the level that you wish SQL statements to be logged at.
+=item C<$StatementLog>,
+
+RT has rudimentary SQL statement logging support if you have
+DBIx-SearchBuilder 1.31_1 or higher; simply set C<$StatementLog> to be
+the level that you wish SQL statements to be logged at.
+
+=cut
+
 Set($StatementLog, undef);
 
-# }}}
+=back
+
+=head1 Web Interface Configuration
+
+=over 4
+
+=item C<$WebDefaultStylesheet>
+
+This determines the default stylesheet the RT web interface will use.
+RT ships with two valid values by default:
 
-# {{{ Web interface configuration
+  3.5-default     The totally new, default layout for RT 3.5
+  3.4-compat      A 3.4 compatibility stylesheet to make RT 3.5 look
+                  (mostly) like 3.4
+
+This value actually specifies a directory in F<share/html/NoAuth/css/>
+from which RT will try to load the file main.css (which should
+ at import any other files the stylesheet needs).  This allows you to
+easily and cleanly create your own stylesheets to apply to RT.  This
+option can be overridden by users in their preferences.
+
+=cut
 
-# This determines the default stylesheet the RT web interface will use.
-# RT ships with two valid values by default:
-#
-#   3.5-default     The totally new, default layout for RT 3.5
-#   3.4-compat      A 3.4 compatibility stylesheet to make RT 3.5 look
-#                   (mostly) like 3.4
-#
-# This value actually specifies a directory in share/html/NoAuth/css/
-# from which RT will try to load the file main.css (which should
-# @import any other files the stylesheet needs).  This allows you to
-# easily and cleanly create your own stylesheets to apply to RT.  This
-# option can be overridden by users in their preferences.
 Set($WebDefaultStylesheet, 'web2');
 
-# Define the directory name to be used for images in rt web
-# documents.
+=item C<$WebPath>
+
+If you're putting the web ui somewhere other than at the root of
+your server, you should set C<$WebPath> to the path you'll be 
+serving RT at.
 
-# If you're putting the web ui somewhere other than at the root of
-# your server, you should set $WebPath to the path you'll be 
-# serving RT at.
-# $WebPath requires a leading / but no trailing /.
-#
-# In most cases, you should leave $WebPath set to '' (an empty value).
+C<$WebPath> requires a leading / but no trailing /.
+
+In most cases, you should leave C<$WebPath> set to '' (an empty value).
+
+=cut
 
 Set($WebPath, "");
 
-# If we're running as a superuser, run on port 80
-# Otherwise, pick a high port for this user.
+=item C<$WebPort>
+
+If we're running as a superuser, run on port 80
+Otherwise, pick a high port for this user.
+
+=cut
 
 Set($WebPort, 80);# + ($< * 7274) % 32766 + ($< && 1024));
 
-# you know what domain name is, right? ;)
+=item C<$WebDomain>
+
+you know what domain name is, right? ;)
+
+=cut
+
 Set( $WebDomain, 'localhost' );
 
-# This is the Scheme, server and port for constructing urls to webrt
-# $WebBaseURL doesn't need a trailing /
+=item C<$WebBaseURL>, C<$WebURL>
+
+This is the Scheme, server and port for constructing urls to webrt
+C<$WebBaseURL> doesn't need a trailing /
+
+=cut
 
 Set($WebBaseURL, 'http://' . RT->Config->Get('WebDomain') . ':' . RT->Config->Get('WebPort'));
 
 Set($WebURL, RT->Config->Get('WebBaseURL') . RT->Config->Get('WebPath') . "/");
 
-# $WebImagesURL points to the base URL where RT can find its images.
+=item C<$WebImagesURL>
+
+C<$WebImagesURL> points to the base URL where RT can find its images.
+Define the directory name to be used for images in rt web
+documents.
+
+=cut
 
 Set($WebImagesURL, RT->Config->Get('WebPath') . "/NoAuth/images/");
 
-# $LogoURL points to the URL of the RT logo displayed in the web UI
+=item C<$LogoURL>
+
+C<$LogoURL> points to the URL of the RT logo displayed in the web UI
+
+=cut
 
 Set($LogoURL, $Config->Get('WebImagesURL') . "bplogo.gif");
 
-# WebNoAuthRegex - What portion of RT's URLspace should not require
-# authentication.
+=item C<$WebNoAuthRegex>
+
+What portion of RT's URLspace should not require authentication.
+
+=cut
+
 Set($WebNoAuthRegex, qr{^ (?:/+NoAuth/ | /+REST/\d+\.\d+/NoAuth/) }x );
 
-# For message boxes, set the entry box width, height and what type of
-# wrapping to use.  These options can be overridden by users in their
-# preferences.
-#
-# Default width: 72, height: 15
+=item C<$MessageBoxWidth>, C<$MessageBoxHeight>
+
+For message boxes, set the entry box width, height and what type of
+wrapping to use.  These options can be overridden by users in their
+preferences.
+
+Default width: 72, height: 15
+
+=cut
+
 Set($MessageBoxWidth, 72);
 Set($MessageBoxHeight, 15);
 
-# Default wrapping: "HARD"  (choices "SOFT", "HARD")
+=item C<$MessageBoxWrap>
+
+Default wrapping: "HARD"  (choices "SOFT", "HARD")
+
+=cut
+
 Set($MessageBoxWrap, "HARD");
 
-# Should "rich text" editing be enabled? This option lets your users send html email messages from the web interface.
+=item C<$MessageBoxRichText>
+
+Should "rich text" editing be enabled? This option lets your users send html email messages from the web interface.
+
+=cut
+
 Set($MessageBoxRichText, 1);
 
-# Support implicit links in WikiText custom fields?  A true value
-# causes InterCapped or ALLCAPS words in WikiText fields to
-# automatically become links to searches for those words.  If used on
-# RTFM articles, it links to the RTFM article with that name.
+=item C<$WikiImplicitLinks>
+
+Support implicit links in WikiText custom fields?  A true value
+causes InterCapped or ALLCAPS words in WikiText fields to
+automatically become links to searches for those words.  If used on
+RTFM articles, it links to the RTFM article with that name.
+
+=cut
+
 Set($WikiImplicitLinks, 0);
 
-# if TrustHTMLAttachments is not defined, we will display them
-# as text. This prevents malicious HTML and javascript from being
-# sent in a request (although there is probably more to it than that)
-Set($TrustHTMLAttachments , undef);
-
-# Should RT redistribute correspondence that it identifies as
-# machine generated? A true value will do so; setting this to '0'
-# will cause no such messages to be redistributed.
-# You can also use 'privileged' (the default), which will redistribute
-# only to privileged users. This helps to protect against malformed
-# bounces and loops caused by autocreated requestors with bogus addresses.
+=item C<$TrustHTMLAttachments>
+
+if C<TrustHTMLAttachments> is not defined, we will display them
+as text. This prevents malicious HTML and javascript from being
+sent in a request (although there is probably more to it than that)
+
+=cut
+
+Set($TrustHTMLAttachments, undef);
+
+=item C<$RedistributeAutoGeneratedMessages>
+
+Should RT redistribute correspondence that it identifies as
+machine generated? A true value will do so; setting this to '0'
+will cause no such messages to be redistributed.
+You can also use 'privileged' (the default), which will redistribute
+only to privileged users. This helps to protect against malformed
+bounces and loops caused by autocreated requestors with bogus addresses.
+
+=cut
+
 Set($RedistributeAutoGeneratedMessages, 'privileged');
 
-# If PreferRichText is set to a true value, RT will show HTML/Rich text
-# messages in preference to their plaintext alternatives. RT "scrubs" the 
-# html to show only a minimal subset of HTML to avoid possible contamination
-# by cross-site-scripting attacks.
+=item C<$PreferRichText>
+
+If C<$PreferRichText> is set to a true value, RT will show HTML/Rich text
+messages in preference to their plaintext alternatives. RT "scrubs" the 
+html to show only a minimal subset of HTML to avoid possible contamination
+by cross-site-scripting attacks.
+
+=cut
+
 Set($PreferRichText, undef);
 
-# If $WebExternalAuth is defined, RT will defer to the environment's
-# REMOTE_USER variable.
+=item C<$WebExternalAuth>
+
+If C<$WebExternalAuth> is defined, RT will defer to the environment's
+REMOTE_USER variable.
+
+=cut
 
-Set($WebExternalAuth , undef);
+Set($WebExternalAuth, undef);
 
-# If $WebFallbackToInternalAuth is undefined, the user is allowed a chance
-# of fallback to the login screen, even if REMOTE_USER failed.
+=item C<$WebFallbackToInternalAuth>
+
+If C<$WebFallbackToInternalAuth> is undefined, the user is allowed a chance
+of fallback to the login screen, even if REMOTE_USER failed.
+
+=cut
 
 Set($WebFallbackToInternalAuth , undef);
 
-# $WebExternalGecos means to match 'gecos' field as the user identity);
-# useful with mod_auth_pwcheck and IIS Integrated Windows logon.
+=item C<$WebExternalGecos>
+
+C<$WebExternalGecos> means to match 'gecos' field as the user identity);
+useful with mod_auth_pwcheck and IIS Integrated Windows logon.
+
+=cut
 
 Set($WebExternalGecos , undef);
 
-# $WebExternalAuto will create users under the same name as REMOTE_USER
-# upon login, if it's missing in the Users table.
+=item C<$WebExternalAuto>
+
+C<$WebExternalAuto> will create users under the same name as REMOTE_USER
+upon login, if it's missing in the Users table.
+
+=cut
 
 Set($WebExternalAuto , undef);
 
-# If $WebExternalAuto is true, $AutoCreate will be passed to User's
-# Create method.  Use it to set defaults, such as creating 
-# Unprivileged users with { Privileged => 0 }
-# Must be a hashref of arguments
+=item C<$AutoCreate>
+
+If C<$WebExternalAuto> is true, C<$AutoCreate> will be passed to User's
+Create method.  Use it to set defaults, such as creating 
+Unprivileged users with C<{ Privileged => 0 }>
+( Must be a hashref of arguments )
+
+=cut
 
 Set($AutoCreate, undef);
 
-# $WebSessionClass is the class you wish to use for managing Sessions.
-# It defaults to use your SQL database, but if you are using MySQL 3.x and
-# plans to use non-ascii Queue names, uncomment and add this line to
-# RT_SiteConfig.pm will prevent session corruption.
+=item C<$WebSessionClass>
+
+C<$WebSessionClass> is the class you wish to use for managing Sessions.
+It defaults to use your SQL database, but if you are using MySQL 3.x and
+plans to use non-ascii Queue names, uncomment and add this line to
+F<RT_SiteConfig.pm> will prevent session corruption.
+
+=cut
 
 # Set($WebSessionClass , 'Apache::Session::File');
 
-# By default, RT's user sessions persist until a user closes his or her 
-# browser. With the $AutoLogoff option you can setup session lifetime in 
-# minutes. A user will be logged out if he or she doesn't send any requests 
-# to RT for the defined time.
+=item C<$AutoLogoff>
+
+By default, RT's user sessions persist until a user closes his or her 
+browser. With the C<$AutoLogoff> option you can setup session lifetime in 
+minutes. A user will be logged out if he or she doesn't send any requests 
+to RT for the defined time.
+
+=cut
 
 Set($AutoLogoff, 0);
 
-# By default, RT's session cookie isn't marked as "secure" Some web browsers 
-# will treat secure cookies more carefully than non-secure ones, being careful
-# not to write them to disk, only send them over an SSL secured connection 
-# and so on. To enable this behaviour, set # $WebSecureCookies to a true value. 
-# NOTE: You probably don't want to turn this on _unless_ users are only connecting
-# via SSL encrypted HTTP connections.
+=item C<$WebSecureCookies>
+
+By default, RT's session cookie isn't marked as "secure" Some web browsers 
+will treat secure cookies more carefully than non-secure ones, being careful
+not to write them to disk, only send them over an SSL secured connection 
+and so on. To enable this behaviour, set C<$WebSecureCookies> to a true value. 
+NOTE: You probably don't want to turn this on _unless_ users are only connecting
+via SSL encrypted HTTP connections.
+
+=cut
 
 Set($WebSecureCookies, 0);
 
+=item C<$WebFlushDbCacheEveryRequest>
+
+By default, RT clears its database cache after every page view.
+This ensures that you've always got the most current information 
+when working in a multi-process (mod_perl or FastCGI) Environment
+Setting C<$WebFlushDbCacheEveryRequest> to '0' will turn this off,
+which will speed RT up a bit, at the expense of a tiny bit of data 
+accuracy.
 
-# By default, RT clears its database cache after every page view.
-# This ensures that you've always got the most current information 
-# when working in a multi-process (mod_perl or FastCGI) Environment
-# Setting $WebFlushDbCacheEveryRequest to '0' will turn this off,
-# which will speed RT up a bit, at the expense of a tiny bit of data 
-# accuracy.
+=cut
 
 Set($WebFlushDbCacheEveryRequest, '1');
 
 
-# $MaxInlineBody is the maximum attachment size that we want to see
-# inline when viewing a transaction.  RT will inline any text if value
-# is undefined or 0.  This option can be overridden by users in their
-# preferences.
+=item C<$MaxInlineBody>
+
+C<$MaxInlineBody> is the maximum attachment size that we want to see
+inline when viewing a transaction.  RT will inline any text if value
+is undefined or 0.  This option can be overridden by users in their
+preferences.
+
+=cut
 
 Set($MaxInlineBody, 12000);
 
-# $DefaultSummaryRows is default number of rows displayed in for search
-# results on the frontpage.
+=item C<$DefaultSummaryRows>
+
+C<$DefaultSummaryRows> is default number of rows displayed in for search
+results on the frontpage.
+
+=cut
 
 Set($DefaultSummaryRows, 10);
 
-# By default, RT shows newest transactions at the bottom of the ticket
-# history page, if you want see them at the top set this to '0'.  This
-# option can be overridden by users in their preferences.
+=item C<$OldestTransactionsFirst>
+
+By default, RT shows newest transactions at the bottom of the ticket
+history page, if you want see them at the top set this to '0'.  This
+option can be overridden by users in their preferences.
+
+=cut
 
 Set($OldestTransactionsFirst, '1');
 
-# By default, RT shows images attached to incoming (and outgoing) ticket updates
-# inline. Set this variable to 0 if you'd like to disable that behaviour
+=item C<$ShowTransactionImages>
+
+By default, RT shows images attached to incoming (and outgoing) ticket updates
+inline. Set this variable to 0 if you'd like to disable that behaviour
+
+=cut
 
 Set($ShowTransactionImages, 1);
 
 
-# By default, RT will prompt users when there are new, unread messages on tickets they are viewing
-# Set ShowUnreadMessageNotifications to a false value to disable this feature.
+=item C<$ShowUnreadMessageNotifications>
+
+By default, RT will prompt users when there are new, unread messages on
+tickets they are viewing.
+
+Set C<$ShowUnreadMessageNotifications> to a false value to disable this feature.
+
+=cut
 
 Set($ShowUnreadMessageNotifications, 1);
 
 
-# $HomepageComponents is an arrayref of allowed components on a user's
-# customized homepage ("RT at a glance").
+=item C<$HomepageComponents>
+
+C<$HomepageComponents> is an arrayref of allowed components on a user's
+customized homepage ("RT at a glance").
+
+=cut
 
 Set($HomepageComponents, [qw(QuickCreate Quicksearch MyAdminQueues MySupportQueues MyReminders RefreshHomepage)]);
 
-# @MasonParameters is the list of parameters for the constructor of
-# HTML::Mason's Apache or CGI Handler.  This is normally only useful
-# for debugging, eg. profiling individual components with:
-#     use MasonX::Profiler; # available on CPAN
-#     @MasonParameters = (preamble => 'my $p = MasonX::Profiler->new($m, $r);');
+=item C<@MasonParameters>
 
-Set(@MasonParameters, ());
+C<@MasonParameters> is the list of parameters for the constructor of
+HTML::Mason's Apache or CGI Handler.  This is normally only useful
+for debugging, eg. profiling individual components with:
 
-# Net::Server (rt-server) configuration {{{
-# The absolute minimum and maximum number of servers that will be created to
-# handle requests. Having multiple servers means that serving a slow page will
-# affect other users less.
-Set($StandaloneMinServers, 1);
-Set($StandaloneMaxServers, 1);
+    use MasonX::Profiler; # available on CPAN
+    @MasonParameters = (preamble => 'my $p = MasonX::Profiler->new($m, $r);');
 
-# These next two options can be used to scale up and down the number of servers
-# to adjust to load. These two options will respect the $StandaloneMinServers
-# and $StandaloneMaxServers options.
-Set($StandaloneMinSpareServers, 0);
-Set($StandaloneMaxSpareServers, 0);
+=cut
 
-# This sets the absolute maximum number of requests a single server will serve.
-# Setting this would be useful if, for example, memory usage slowly crawls up
-# every hit.
-#Set($StandaloneMaxRequests, 50);
+Set(@MasonParameters, ());
 
-# %NetServerOptions is a hash of additional options to use for
-# L<Net::Server/DEFAULT ARGUMENTS>. For example, you could set
-# reverse_lookups to get the hostnames for all users with:
-# Set(%NetServerOptions, (reverse_lookups => 1));
-Set(%NetServerOptions, ());
-# }}}
+=item C<$DefaultSearchResultFormat>
+
+C<$DefaultSearchResultFormat> is the default format for RT search results
+
+=cut
 
-# $DefaultSearchResultFormat is the default format for RT search results
 Set ($DefaultSearchResultFormat, qq{
    '<B><A HREF="__WebPath__/Ticket/Display.html?id=__id__">__id__</a></B>/TITLE:#',
    '<B><A HREF="__WebPath__/Ticket/Display.html?id=__id__">__Subject__</a></B>/TITLE:Subject',
@@ -643,169 +1033,339 @@
    '<small>__LastUpdatedRelative__</small>',
    '<small>__TimeLeft__</small>'});
 
-# If $SuppressInlineTextFiles is set to a true value, then uploaded
-# text files (text-type attachments with file names) are prevented
-# from being displayed in-line when viewing a ticket's history.
+
+=item C<$SuppressInlineTextFiles>
+
+If C<$SuppressInlineTextFiles> is set to a true value, then uploaded
+text files (text-type attachments with file names) are prevented
+from being displayed in-line when viewing a ticket's history.
+
+=cut
 
 Set($SuppressInlineTextFiles, undef);
 
-# If $DontSearchFileAttachments is set to a true value, then uploaded
-# files (attachments with file names) are not searched during full-content
-# ticket searches.
+=item C<DontSearchFileAttachments>
+
+If C<$DontSearchFileAttachments> is set to a true value, then uploaded
+files (attachments with file names) are not searched during full-content
+ticket searches.
+
+=cut
 
 Set($DontSearchFileAttachments, undef);
 
-# The GD module (which RT uses for graphs) uses a builtin font that doesn't
-# have full Unicode support. You can use a particular TrueType font by setting
-# $ChartFont to the absolute path of that font. Your GD library must have
-# support for TrueType fonts to use this option.
+=item C<$ChartFont>
+
+The L<GD> module (which RT uses for graphs) uses a builtin font that doesn't
+have full Unicode support. You can use a particular TrueType font by setting
+$ChartFont to the absolute path of that font. Your GD library must have
+support for TrueType fonts to use this option.
+
+=cut
 
 Set($ChartFont, undef);
 
 
-# MakeClicky detects various formats of data in headers and email
-# messages, and extends them with supporting links.  By default, RT
-# provides two formats:
-#
-# * 'httpurl': detects http:// and https:// URLs and adds '[Open URL]'
-#   link after the URL.
-#
-# * 'httpurl_overwrite': also detects URLs as 'httpurl' format, but
-#   replace URL with link and *adds spaces* into text if it's longer
-#   then 30 chars. This allow browser to wrap long URLs and avoid
-#   horizontal scrolling.
-#
-# See html/Elements/MakeClicky for documentation on how to add your own.
+=item C<@Active_MakeClicky>
+
+MakeClicky detects various formats of data in headers and email
+messages, and extends them with supporting links.  By default, RT
+provides two formats:
+
+* 'httpurl': detects http:// and https:// URLs and adds '[Open URL]'
+  link after the URL.
+
+* 'httpurl_overwrite': also detects URLs as 'httpurl' format, but
+  replace URL with link and *adds spaces* into text if it's longer
+  then 30 chars. This allow browser to wrap long URLs and avoid
+  horizontal scrolling.
+
+See F<share/html/Elements/MakeClicky> for documentation on how to add your own.
+
+=cut
+
 Set(@Active_MakeClicky, qw());
 
-# }}}
+=back
+
+=head1 L<Net::Server> (rt-server) Configuration
+
+=over 4
+
+=item C<$StandaloneMinServers>, C<$StandaloneMaxServers>
+
+The absolute minimum and maximum number of servers that will be created to
+handle requests. Having multiple servers means that serving a slow page will
+affect other users less.
+
+=cut
+
+Set($StandaloneMinServers, 1);
+Set($StandaloneMaxServers, 1);
+
+=item C<$StandaloneMinSpareServers>, C<$StandaloneMaxSpareServers>
+
+These next two options can be used to scale up and down the number of servers
+to adjust to load. These two options will respect the C<$StandaloneMinServers
+> and C<$StandaloneMaxServers options>.
+
+=cut
+
+Set($StandaloneMinSpareServers, 0);
+Set($StandaloneMaxSpareServers, 0);
+
+=item C<$StandaloneMaxRequests>
+
+This sets the absolute maximum number of requests a single server will serve.
+Setting this would be useful if, for example, memory usage slowly crawls up
+every hit.
+
+=cut
+
+#Set($StandaloneMaxRequests, 50);
+
+=item C<%NetServerOptions>
+
+C<%NetServerOptions> is a hash of additional options to use for
+L<Net::Server/DEFAULT ARGUMENTS>. For example, you could set
+reverse_lookups to get the hostnames for all users with:
+
+C<Set(%NetServerOptions, (reverse_lookups => 1));>
+
+=cut
+
+Set(%NetServerOptions, ());
+
+=back
 
-# {{{ RT UTF-8 Settings
 
-# An array that contains languages supported by RT's internationalization
-# interface.  Defaults to all *.po lexicons; setting it to qw(en ja) will make
-# RT bilingual instead of multilingual, but will save some memory.
+=head1 UTF-8 Configuration
+
+=over 4
+
+=item C<@LexiconLanguages>
+
+An array that contains languages supported by RT's internationalization
+interface.  Defaults to all *.po lexicons; setting it to C<qw(en ja)> will make
+RT bilingual instead of multilingual, but will save some memory.
+
+=cut
 
 Set(@LexiconLanguages, qw(*));
 
-# An array that contains default encodings used to guess which charset
-# an attachment uses if not specified.  Must be recognized by
-# Encode::Guess.
+=item C<@EmailInputEncodings>
+
+An array that contains default encodings used to guess which charset
+an attachment uses if not specified.  Must be recognized by
+L<Encode::Guess>.
+
+=cut
 
 Set(@EmailInputEncodings, qw(utf-8 iso-8859-1 us-ascii));
 
-# The charset for localized email.  Must be recognized by Encode.
+=item C<$EmailOutputEncoding>
+
+The charset for localized email.  Must be recognized by Encode.
+
+=cut
+
+Set($EmailOutputEncoding, 'utf-8');
+
+
+=back
+
+=head1 Date Handling Configuration
 
-Set($EmailOutputEncoding , 'utf-8');
+=over 4
 
-# }}}
+=item C<$DateTimeFormat>
 
-# {{{ RT Date Handling Options
+You can choose date and time format.  See "Output formatters"
+section in perldoc F<lib/RT/Date.pm> for more options.  This option can
+be overridden by users in their preferences.
+Some examples:
+
+C<Set($DateTimeFormat, { Format => 'ISO', Seconds => 0 });>
+C<Set($DateTimeFormat, 'RFC2822');>
+C<Set($DateTimeFormat, { Format => 'RFC2822', Seconds => 0, DayOfWeek => 0 });>
+
+=cut
 
-# You can choose date and time format.  See "Output formatters"
-# section in perldoc lib/RT/Date.pm for more options.  This option can
-# be overridden by users in their preferences.
-# Some examples:
-#Set($DateTimeFormat, { Format => 'ISO', Seconds => 0 });
-#Set($DateTimeFormat, 'RFC2822');
-#Set($DateTimeFormat, { Format => 'RFC2822', Seconds => 0, DayOfWeek => 0 });
 Set($DateTimeFormat, 'DefaultFormat');
 
 # Next two options are for Time::ParseDate
-# Set this to 1 if your local date convention looks like "dd/mm/yy"
-# instead of "mm/dd/yy".
+
+=item C<$DateDayBeforeMonth>
+
+Set this to 1 if your local date convention looks like "dd/mm/yy"
+instead of "mm/dd/yy".
+
+=cut
 
 Set($DateDayBeforeMonth , 1);
 
-# Should an unspecified day or year in a date refer to a future or a
-# past value? For example, should a date of "Tuesday" default to mean
-# the date for next Tuesday or last Tuesday? Should the date "March 1"
-# default to the date for next March or last March?
-# Set AmbiguousDayInPast for the last date, or AmbiguousDayInFuture for the
-# next date. The default is usually good.
+=item C<$AmbiguousDayInPast>, C<$AmbiguousDayInFuture>
+
+Should an unspecified day or year in a date refer to a future or a
+past value? For example, should a date of "Tuesday" default to mean
+the date for next Tuesday or last Tuesday? Should the date "March 1"
+default to the date for next March or last March?
+
+Set $<AmbiguousDayInPast> for the last date, or $<$AmbiguousDayInFuture> for the
+next date.
+
+The default is usually good.
+
+=cut
 
 Set($AmbiguousDayInPast, 0);
 Set($AmbiguousDayInFuture, 0);
 
-# }}}
+=back
+
+=head1 Miscellaneous Configuration
+
+=over 4
+
+=item C<@ActiveStatus>, C<@InactiveStatus>
 
-# {{{ Miscellaneous RT Settings
+You can define new statuses and even reorder existing statuses here.
+WARNING. DO NOT DELETE ANY OF THE DEFAULT STATUSES. If you do, RT
+will break horribly. The statuses you add must be no longer than
+10 characters.
 
-# You can define new statuses and even reorder existing statuses here.
-# WARNING. DO NOT DELETE ANY OF THE DEFAULT STATUSES. If you do, RT
-# will break horribly. The statuses you add must be no longer than
-# 10 characters.
+=cut
 
 Set(@ActiveStatus, qw(new open stalled));
 Set(@InactiveStatus, qw(resolved rejected deleted));
 
-# RT-3.4 backward compatibility setting. Add/Delete Link used to record one
-# transaction and run one scrip. Set this value to 1 if you want
-# only one of the link transactions to have scrips run.
-Set($LinkTransactionsRun1Scrip , 0);
-
-# When this feature is enabled a user needs ModifyTicket rights on both
-# tickets to link them together, otherwise he can have rights on either of
-# them.
+=item C<$LinkTransactionsRun1Scrip>
+
+RT-3.4 backward compatibility setting. Add/Delete Link used to record one
+transaction and run one scrip. Set this value to 1 if you want
+only one of the link transactions to have scrips run.
+
+=cut
+
+Set($LinkTransactionsRun1Scrip, 0);
+
+=item C<$StrictLinkACL>
+
+When this feature is enabled a user needs I<ModifyTicket> rights on both
+tickets to link them together, otherwise he can have rights on either of
+them.
+
+=cut
+
 Set($StrictLinkACL, 1);
 
-# Set $PreviewScripMessages to 1 if the scrips preview on the ticket
-# reply page should include the content of the messages to be sent.
+=item C<$PreviewScripMessages>
+
+Set C<$PreviewScripMessages> to 1 if the scrips preview on the ticket
+reply page should include the content of the messages to be sent.
+
+=cut
+
 Set($PreviewScripMessages, 0);
 
-# Set $UseTransactionBatch to 1 to execute transactions in batches,
-# such that a resolve and comment (for example) would happen
-# simultaneously, instead of as two transactions, unaware of each
-# others' existence.
+=item C<$UseTransactionBatch>
+
+Set C<$UseTransactionBatch> to 1 to execute transactions in batches,
+such that a resolve and comment (for example) would happen
+simultaneously, instead of as two transactions, unaware of each
+others' existence.
+
+=cut
+
 Set($UseTransactionBatch, 0);
 
-# Set @CustomFieldValuesSources to a list of class names which extend
-# RT::CustomFieldValues::External.  This can be used to pull lists of
-# custom field values from external sources at runtime.
+=item C<@CustomFieldValuesSources>
+
+Set C<@CustomFieldValuesSources> to a list of class names which extend
+L<RT::CustomFieldValues::External>.  This can be used to pull lists of
+custom field values from external sources at runtime.
+
+=cut
+
 Set(@CustomFieldValuesSources, ());
 
-# Set $CanonicalizeRedirectURLs to 1 to use $WebURL when redirecting rather
-# than the one we get from %ENV.
-# If you use RT behind a reverse proxy, you almost certainly want to
-# enable this option.
+=item C<$CanonicalizeRedirectURLs>
 
-Set($CanonicalizeRedirectURLs, 0);
+Set C<$CanonicalizeRedirectURLs> to 1 to use $C<WebURL> when redirecting rather
+than the one we get from C<%ENV>.
 
-# }}}
+If you use RT behind a reverse proxy, you almost certainly want to
+enable this option.
 
-# {{{ Development Mode
-#
-# RT comes with a "Development mode" setting. 
-# This setting, as a convenience for developers, turns on 
-# all sorts of development options that you most likely don't want in 
-# production:
-#
-# * Turns off Mason's 'static_source' directive. By default, you can't 
-#   edit RT's web ui components on the fly and have RT magically pick up
-#   your changes. (It's a big performance hit)
-#
-#  * More to come
-#
+=cut
 
-Set($DevelMode, '@RT_DEVEL_MODE@');
+Set($CanonicalizeRedirectURLs, 0);
+
+=item C<$DisableGraphViz>
+
+=cut
 
 Set($DisableGraphViz, do { local $@; eval {require GraphViz; 1;} ? 0 : 1 } );
 
+=item C<$DisableGD>
+
+=cut
+
 Set($DisableGD, do { local $@; eval {require GD; 1;} ? 0 : 1 } );
 
-# }}}
 
-# Reminders
-# Hide links/portlets related to Reminders by setting this to 0
+=item C<$EnableReminders>
+
+Hide links/portlets related to Reminders by setting this to 0
+
+=cut
 
 Set($EnableReminders,1);
 
-# {{{ Deprecated options
+=back
+
+=head1 Development Configuration
+
+=over 4
+
+=item C<$DevelMode>
+
+RT comes with a "Development mode" setting. 
+This setting, as a convenience for developers, turns on 
+all sorts of development options that you most likely don't want in 
+production:
+
+* Turns off Mason's 'static_source' directive. By default, you can't 
+  edit RT's web ui components on the fly and have RT magically pick up
+  your changes. (It's a big performance hit)
+
+ * More to come
 
-# $AlwaysUseBase64 - Encode blobs as base64 in DB (?)
-# $TicketBaseURI - Base URI to tickets in this system; used when loading (?)
-# $UseCodeTickets - This option is exists for backwards compatibility.  Don't use it.
+=cut
+
+Set($DevelMode, '@RT_DEVEL_MODE@');
+
+
+=back
+
+=head1 Deprecated Options
+
+=over 4
+
+=item C<$AlwaysUseBase64>
 
-# }}}
+Encode blobs as base64 in DB (?)
+
+=item C<$TicketBaseURI>
+
+Base URI to tickets in this system; used when loading (?)
+
+=item C<$UseCodeTickets>
+
+This option is exists for backwards compatibility.  Don't use it.
+
+=back
+
+=cut
 
 1;

Modified: rt/branches/3.8-TESTING/lib/RT/Config.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Config.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Config.pm	Mon Jun  9 17:24:04 2008
@@ -109,7 +109,7 @@
         },
     },
     DefaultSummaryRows => {
-        Section         => 'RT at a glance',
+        Section         => 'RT at a glance',    #loc
         Overridable     => 1,
         Widget          => '/Widgets/Form/Integer',
         WidgetArguments => {
@@ -187,7 +187,7 @@
         Overridable     => 1,
         Widget          => '/Widgets/Form/Select',
         WidgetArguments => {
-            Description => 'Default queue',
+            Description => 'Default queue',    #loc
             Callback    => sub {
                 my $ret = { Values => [], ValuesLabel => {}};
                 my $q = new RT::Queues($HTML::Mason::Commands::session{'CurrentUser'});
@@ -207,12 +207,12 @@
             Default     => 'Individual messages',
         Widget          => '/Widgets/Form/Select',
         WidgetArguments => {
-            Description => 'Email delivery',
+            Description => 'Email delivery',    #loc
             Values      => [
-                'Individual messages',
-                'Daily digest',
-                'Weekly digest',
-                'Suspended'
+                'Individual messages',    #loc
+                'Daily digest',           #loc
+                'Weekly digest',          #loc
+                'Suspended'               #loc
             ]
             }
             },

Modified: rt/branches/3.8-TESTING/lib/RT/I18N/fr.po
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/I18N/fr.po	(original)
+++ rt/branches/3.8-TESTING/lib/RT/I18N/fr.po	Mon Jun  9 17:24:04 2008
@@ -5002,7 +5002,7 @@
 
 #: share/html/Prefs/Quicksearch.html:55
 msgid "Select queues to be displayed on the \"RT at a glance\" page"
-msgstr "Sélectionnez les files à afficher dans la page \"RT en un coup d'œil\" page"
+msgstr "Sélectionnez les files à afficher dans la page \"RT en un coup d'œil\""
 
 #: share/html/Admin/Global/Scrip.html:61 share/html/Admin/Global/Scrips.html:59 share/html/Admin/Queues/Scrip.html:69 share/html/Admin/Queues/Scrips.html:75
 msgid "Select scrip"

Modified: rt/branches/3.8-TESTING/lib/RT/Interface/REST.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Interface/REST.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Interface/REST.pm	Mon Jun  9 17:24:04 2008
@@ -63,7 +63,7 @@
     @EXPORT = qw(expand_list form_parse form_compose vpush vsplit);
 }
 
-my $field = '(?i:[a-z][a-z0-9_-]*|C(?:ustom)?F(?:ield)?-(?:[a-z0-9_ -]|\s)+)';
+sub field_spec { '(?i:[a-z][a-z0-9_-]*|C(?:ustom)?F(?:ield)?-(?:[a-z0-9_ :()/-]|\s)+)' }
 
 # WARN: this code is duplicated in bin/rt.in,
 # change both functions at once
@@ -96,6 +96,7 @@
     my @forms = ();
     my @lines = split /\n/, $_[0];
     my ($c, $o, $k, $e) = ("", [], {}, "");
+    my $field = __PACKAGE__->field_spec;
 
     LINE:
     while (@lines) {

Modified: rt/branches/3.8-TESTING/lib/RT/Test.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Test.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Test.pm	Mon Jun  9 17:24:04 2008
@@ -344,6 +344,27 @@
     return $obj;
 }
 
+=head2 load_or_create_custom_field
+
+=cut
+
+sub load_or_create_custom_field {
+    my $self = shift;
+    my %args = ( Disabled => 0, @_ );
+    my $obj = RT::CustomField->new( $RT::SystemUser );
+    if ( $args{'Name'} ) {
+        $obj->LoadByName( Name => $args{'Name'}, Queue => $args{'Queue'} );
+    } else {
+        die "Name is required";
+    }
+    unless ( $obj->id ) {
+        my ($val, $msg) = $obj->Create( %args );
+        die "$msg" unless $val;
+    }
+
+    return $obj;
+}
+
 sub store_rights {
     my $self = shift;
 

Modified: rt/branches/3.8-TESTING/sbin/extract-message-catalog
==============================================================================
--- rt/branches/3.8-TESTING/sbin/extract-message-catalog	(original)
+++ rt/branches/3.8-TESTING/sbin/extract-message-catalog	Mon Jun  9 17:24:04 2008
@@ -121,6 +121,7 @@
     my $line = 1;
     while (m!\G.*?<&\|/l(.*?)&>(.*?)</&>!sg) {
         my ( $vars, $str ) = ( $1, $2 );
+        $vars =~ s/[\n\r]//g;
         $line += ( () = ( $& =~ /\n/g ) );    # cryptocontext!
         $str =~ s/\\'/\'/g;
         #print "STR IS $str\n";

Modified: rt/branches/3.8-TESTING/share/html/Elements/CollectionList
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Elements/CollectionList	(original)
+++ rt/branches/3.8-TESTING/share/html/Elements/CollectionList	Mon Jun  9 17:24:04 2008
@@ -45,61 +45,13 @@
 %# those contributions and any derivatives thereof.
 %# 
 %# END BPS TAGGED BLOCK }}}
-<table border="0" cellspacing="0" cellpadding="1" width="100%" class="collection">
-
-% if ( $ShowHeader ) {
-<& /Elements/CollectionAsTable/Header,
-    %ARGS,
-    Class        => $Class,
-    Format       => \@Format,
-    FormatString => $Format,
-    Order        => \@Order,
-    OrderBy      => \@OrderBy,
-    Rows         => $Rows,
-    Page         => $Page,
-    AllowSorting => $AllowSorting, 
-    BaseURL      => $BaseURL,
-    maxitems     => $maxitems,
-&> 
-% }
-
-<%PERL>
-my ($i, $column_map) = (0, {});
-while ( my $record = $Collection->Next ) {
-    # Every ten rows, flush the buffer and put something on the page.
-    $m->flush_buffer unless ++$i % 10;
-    $m->comp('/Elements/CollectionAsTable/Row',
-        i => $i,
-        Format => \@Format,
-        record => $record,
-        maxitems => $maxitems,
-        ColumnMap => $column_map,
-        Class     => $Class,
-    );
+<%INIT>
+if (!$Collection && $Class eq 'RT::Tickets') {
+    $Collection = RT::Tickets->new( $session{'CurrentUser'} );
+    $Collection->FromSQL($Query);
 }
-</%PERL>
 
-</table>
-
-% if ( $Rows && $ShowNavigation && $TotalFound > $Rows ) {
-<hr />
-% my $oddRows = ($TotalFound && $TotalFound % $Rows == 0 )? 0 : 1;
-<&|/l, $Page, int($TotalFound/$Rows)+$oddRows&>Page [_1] of [_2]</&>
-
-<%PERL>
-my %query_args = map { $_ => $ARGS{$_} } @PassArguments;
-my $prev = $m->comp( '/Elements/QueryString', %query_args, Page => $Page - 1 );
-my $next = $m->comp( '/Elements/QueryString', %query_args, Page => $Page + 1 );
-</%PERL>
-% if ( $Page > 1 ) {
-<a href="<% $BaseURL %><% $prev %>"><&|/l&>Previous Page</&></a>
-% }
-% if ( ($Page * $Rows) < $TotalFound ) {
-<a href="<% $BaseURL %><% $next %>"><&|/l&>Next Page</&></a>
-% }
-% }
-<%INIT>
-my $TotalFound = $Collection->CountAll;
+my $TotalFound = $Collection->CountAll();
 return '' if !$TotalFound && !$ShowEmpty;
 
 if ( @OrderBy ) {
@@ -114,6 +66,8 @@
 }
 
 $Collection->RowsPerPage( $Rows ) if $Rows;
+$Page = 1
+  unless $Page && $Page > 0;    # workaround problems with Page = '' or undef
 $Collection->GotoPage( $Page - 1 ); # SB uses page 0 as the first page
 
 # DisplayFormat lets us use a "temporary" format for display, while 
@@ -140,29 +94,84 @@
     }
 }
 
-my $Class = ref $Collection;
+$Class ||= ref $Collection;
 $Class =~ s/s$//;
 $Class =~ s/:/_/g;
 
+$m->out('<table border="0" cellspacing="0" cellpadding="1"' . 
+	' width="100%" class="' . 
+	($Collection->isa('RT::Tickets') ? 'ticket-list' : 'collection') . '>');
+
+if ( $ShowHeader ) {
+  $m->comp('/Elements/CollectionAsTable/Header',
+	   %ARGS,
+	   Class        => $Class,
+	   Format       => \@Format,
+	   FormatString => $Format,
+	   Order        => \@Order,
+	   OrderBy      => \@OrderBy,
+	   Rows         => $Rows,
+	   Page         => $Page,
+	   AllowSorting => $AllowSorting, 
+	   BaseURL      => $BaseURL,
+	   maxitems     => $maxitems,
+	   );
+}
+
+my ($i, $column_map) = (0, {});
+while ( my $record = $Collection->Next ) {
+    # Every ten rows, flush the buffer and put something on the page.
+    $m->flush_buffer unless ++$i % 10;
+    $m->comp('/Elements/CollectionAsTable/Row',
+        i => $i,
+        Format => \@Format,
+        record => $record,
+        maxitems => $maxitems,
+        ColumnMap => $column_map,
+        Class     => $Class,
+    );
+}
+
+$m->out('</table>');
+if ( $Rows && $ShowNavigation && $TotalFound > $Rows ) {
+  $m->out('<hr />');
+  my $oddRows = ($TotalFound && $TotalFound % $Rows == 0 )? 0 : 1;
+  my $pages = int( $TotalFound / $Rows ) + $oddRows;
+  $pages = 1 if $pages < 1;
+
+
+  my %query_args = map { $_ => $ARGS{$_} } @PassArguments;
+  $m->comp(
+        '/Elements/CollectionListPaging',
+        BaseURL     => $BaseURL,
+        Rows        => $Rows,
+        TotalFound  => $TotalFound,
+        CurrentPage => $Page,
+        Pages       => $pages,
+        URLParams   => \%query_args
+    );
+
+}
+
 </%INIT>
 <%ARGS>
+$Class         => ''
 $Collection    => undef
-
 $Format        => undef
 $DisplayFormat => undef
-
 @Order         => ()
 @OrderBy       => ()
 
-$Rows          => undef
+$Rows          => 1
 $Page          => 1
-
 $Title         => loc('Ticket Search')
 $BaseURL       => RT->Config->Get('WebPath') . $m->request_comp->path .'?'
- at PassArguments => qw(Query Format Rows Page Order OrderBy)
+ at PassArguments => qw( Query Format Rows Page Order OrderBy)
 
 $AllowSorting   => 0
 $ShowNavigation => 1
 $ShowHeader     => 1
 $ShowEmpty      => 0
+$IsTicket => 0
+$Query => 0
 </%ARGS>

Modified: rt/branches/3.8-TESTING/share/html/Elements/CollectionListPaging
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Elements/CollectionListPaging	(original)
+++ rt/branches/3.8-TESTING/share/html/Elements/CollectionListPaging	Mon Jun  9 17:24:04 2008
@@ -8,11 +8,14 @@
 </%ARGS>
 
 <%INIT>
+if ($Pages == 1) {
+  $m->out(loc('Page 1 of 1'));
+}
+else{
 $m->out(loc('Page') . ' ');
-
 my $prev = $m->comp(
 		    '/Elements/QueryString',
-		    @$URLParams,
+		    %$URLParams,
 		    Page    => ( $CurrentPage - 1 )
 		   );
 my %show;
@@ -23,7 +26,7 @@
 for my $number (1..$Pages){
   if ($show{$number}){
     $dots = undef;
-    my $qs = $m->comp('/Elements/QueryString', @$URLParams, Page => $number);
+    my $qs = $m->comp('/Elements/QueryString', %$URLParams, Page => $number);
     $m->out(  qq{<a href="$BaseURL$qs">}
 	      . (($number == $CurrentPage)  ? "<em>$number</em>" : $number) 
 	      . qq{</a> });
@@ -36,13 +39,14 @@
 
 my $next = $m->comp(
 		    '/Elements/QueryString',
-		    @$URLParams,
+		    %$URLParams,
 		    Page    => ( $CurrentPage + 1 )
 		   );
+if ($CurrentPage > 1) {
+  $m->out(qq{<a href="$BaseURL$prev">}.loc('Previous ').qq{</a>});
+}
+if (($CurrentPage * $Rows) < $TotalFound) {
+  $m->out(qq{<a href="$BaseURL$next">}.loc('Next').qq{</a>});
+}
+}
 </%INIT>
-% if ($CurrentPage > 1) {
-<a href="<%$BaseURL%><%$prev%>"><&|/l&>Previous</&></a>
-% }
-% if (($CurrentPage * $Rows) < $TotalFound) {
-<a href="<%$BaseURL%><%$next%>"><&|/l&>Next</&></a>
-% }

Modified: rt/branches/3.8-TESTING/share/html/Elements/TicketList
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Elements/TicketList	(original)
+++ rt/branches/3.8-TESTING/share/html/Elements/TicketList	Mon Jun  9 17:24:04 2008
@@ -45,164 +45,23 @@
 %# those contributions and any derivatives thereof.
 %# 
 %# END BPS TAGGED BLOCK }}}
-<table border="0" cellspacing="0" cellpadding="1" width="100%" class="ticket-list">
-
-% if ($ShowHeader) {
-<& /Elements/CollectionAsTable/Header, 
-    Format => \@Format, 
-    FormatString => $Format,
-    AllowSorting => $AllowSorting, 
-    Order => $Order, 
-    Query => $Query,
-    Rows => $Rows,
-    Page => $Page,
-    OrderBy => $OrderBy , 
-    BaseURL => $BaseURL,
-    maxitems => $maxitems &> 
-% }
-<%PERL>
-my ($i, $column_map) = (0, {});
-while ( my $record = $Collection->Next ) {
-    $i++;
-    # Every ten rows, flush the buffer and put something on the page.
-    $m->flush_buffer unless $i % 10;
-    $m->comp('/Elements/CollectionAsTable/Row',
-        i => $i,
-        Format => \@Format,
-        record => $record,
-        maxitems => $maxitems,
-        ColumnMap => $column_map,
-    );
-}
-</%PERL>
-
-</table>
-
-%if ($Rows && $ShowNavigation) {
-<hr />
-% my $oddRows = ($TotalFound && $TotalFound % $Rows == 0 )? 0 : 1;
-% my $pages = int($TotalFound/$Rows)+$oddRows;
-% $pages = 1 if $pages < 1;
-<&|/l, $Page, $pages &>Page </&>
-
-<%perl>
-my $prev = $m->comp(
-    '/Elements/QueryString',
-    Query   => $Query,
-    Format  => $Format,
-    Rows    => $Rows,
-    OrderBy => $OrderBy,
-    Order   => $Order,
-    Page    => ( $Page - 1 )
-);
-
-my %show;
-$show{1} = 1;
-$show{$_} = 1 for (($Page - 2)..($Page + 2));
-$show{$pages} = 1;
-my $dots;
-for my $number (1..$pages){
-  if ($show{$number}){
-    $dots = undef;
-    if ($number == $Page){
-      $number = ("<em>$number </em>");
-    }	
-    my $num = $m->comp(
-		       '/Elements/QueryString',
-		       Query   => $Query,
-		       Format  => $Format,
-		       Rows    => $Rows,
-		       OrderBy => $OrderBy,
-		       Order   => $Order,
-		       Page    => $number
-		      );
-    $m->out("<a href=\"$BaseURL$num\">$number </a>");
-  }
-  elsif (not $dots){
-    $dots = 1;
-    $m->out(" ... ");
-  }
-}
-
-my $next = $m->comp(
-    '/Elements/QueryString',
-    Query   => $Query,
-    Format  => $Format,
-    Rows    => $Rows,
-    OrderBy => $OrderBy,
-    Order   => $Order,
-    Page    => ( $Page + 1 )
-);
-</%perl>
-% if ($Page > 1) {
-<a href="<%$BaseURL%><%$prev%>"><&|/l&>Previous</&></a>
-% }
-% if (($Page * $Rows) < $TotalFound) {
-<a href="<%$BaseURL%><%$next%>"><&|/l&>Next</&></a>
-% }
-% }
 <%INIT>
-unless ($Collection) {
-        $Collection = RT::Tickets->new( $session{'CurrentUser'} );
-        $Collection->FromSQL( $Query );
-}
-
-my $TotalFound =  $Collection->CountAll();
-return '' if !$TotalFound && !$ShowEmpty;
-
-if ($OrderBy =~ /\|/) {
-    # Multiple Sorts
-    my @OrderBy = split /\|/,$OrderBy;
-    my @Order = split /\|/,$Order;
-    $Collection->OrderByCols(
-        map { { FIELD => $OrderBy[$_], ORDER => $Order[$_] } }
-        ( 0 .. $#OrderBy )
-    );
-} else {
-    $Collection->OrderBy(FIELD => $OrderBy, ORDER => $Order); 
-}
-
-$Collection->RowsPerPage( $Rows ) if $Rows;
-$Page = 1 unless $Page && $Page > 0; # workaround problems with Page = '' or undef
-$Collection->GotoPage( $Page - 1 ); # SB uses page 0 as the first page
-
-# DisplayFormat lets us use a "temporary" format for display, while 
-# still using our original format for next/prev page links.
-# bulk update uses this feature to add checkboxes
-
-$DisplayFormat ||= $Format;
-
-# Scrub the html of the format string to remove any potential nasties.
-$Format = $m->comp('/Elements/ScrubHTML', Content => $Format);
-$DisplayFormat = $m->comp('/Elements/ScrubHTML', Content => $DisplayFormat);
-
-my @Format = $m->comp('/Elements/CollectionAsTable/ParseFormat', Format => $DisplayFormat);
-
-# Find the maximum number of items in any row, so we can pad the table.
-my ($maxitems, $item) = (0, 0);
-foreach my $col (@Format) {
-    $item++;
-    if ( $col->{title} && ($col->{title} eq 'NEWLINE') ) {
-        $item = 0;
-    }
-    else {
-        $maxitems = $item if $item > $maxitems;
-    }
-}
-
+$m->comp(
+	 '/Elements/CollectionList',
+	 %ARGS,
+	 Class => 'RT::Tickets'
+	);
 </%INIT>
 <%ARGS>
 $Collection => undef
-
 $Query => undef
 $Format => RT->Config->Get('DefaultSearchResultFormat')
 $DisplayFormat => undef
 $Order => undef
 $OrderBy => undef
 $Rows => 50
-
 $Page => 1
-$Title => 'Ticket Search'
+$Title => loc('Ticket Search')
 $BaseURL => undef
 $AllowSorting => undef
 $ShowNavigation => 1

Modified: rt/branches/3.8-TESTING/share/html/Install/Emails.html
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Install/Emails.html	(original)
+++ rt/branches/3.8-TESTING/share/html/Install/Emails.html	Mon Jun  9 17:24:04 2008
@@ -49,6 +49,14 @@
 
 <& /Elements/ListActions, actions => \@results &>
 
+% if ( @notes ) {
+<ul>
+% for ( @notes ) {
+<li><% $_ %></li>
+% }
+</ul>
+% }
+
 <form method="post">
 <& /Widgets/BulkEdit, Types => \@Types,Meta => $RT::Installer->{Meta},
     CurrentValue => RT::Installer->CurrentValues(@Types) &>
@@ -60,8 +68,12 @@
 </&>
 <%init>
 my @results;
+my @notes = ( 
+'OwnerEmail is the address of a human who manages RT.',
+'Correspond Address and Comment Address are the default addresses that will be listed in From: and Reply-To: headers of correspondence and comment mail.',
+        );
 
-my @Types = qw/MaxAttachmentSize OwnerEmail CommentAddress CorrespondAddress/;
+my @Types = qw/OwnerEmail CommentAddress CorrespondAddress/;
 
 if ( $Run ) {
 
@@ -74,12 +86,6 @@
 'Install/Basics.html');
     }
 
-    for ( qw/MaxAttachmentSize/ ) {
-        if ( $ARGS{$_} && $ARGS{$_} !~ /^\d+$/ ) {
-            push @results, "Invalid $_: it should be a number";
-        }
-    }
-
     unless ( @results ) {
         RT::Interface::Web::Redirect(RT->Config->Get('WebURL') .
 'Install/Sendmail.html');

Modified: rt/branches/3.8-TESTING/share/html/Install/Sendmail.html
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Install/Sendmail.html	(original)
+++ rt/branches/3.8-TESTING/share/html/Install/Sendmail.html	Mon Jun  9 17:24:04 2008
@@ -49,10 +49,6 @@
 
 <& /Elements/ListActions, actions => \@results &>
 
-<div>
-Options below 'Mail command' only take effect if Mail command is 'sendmail' or 'sendmailpipe'
-</div>
-<hr />
 <form method="post">
 <& /Widgets/BulkEdit, Types => \@Types,Meta => $RT::Installer->{Meta},
     CurrentValue => RT::Installer->CurrentValues(@Types) &>
@@ -60,13 +56,13 @@
 <input type="hidden" name="Run" value="1">
 <& /Elements/Submit, Label => $RT::Installer->{DatabaseAction} eq 'none' ?
 loc('Next: Finish') : loc('Next: Initialize Database'), Back => 1,
-    BackLabel => loc('Back: Customize Emails') &>
+    BackLabel => loc('Back: Customize Email Addresses') &>
 </form>
 </&>
 <%init>
 my @results;
 
-my @Types = qw/SendmailArguments SendmailBounceArguments SendmailPath/;
+my @Types = qw/MaxAttachmentSize SendmailArguments SendmailBounceArguments SendmailPath/;
 
 if ( $Run ) {
 
@@ -79,6 +75,12 @@
 'Install/Emails.html');
     }
 
+    for ( qw/MaxAttachmentSize/ ) {
+        if ( $ARGS{$_} && $ARGS{$_} !~ /^\d+$/ ) {
+            push @results, "Invalid $_: it should be a number";
+        }
+    }
+
     unless ( @results ) {
 
         my ( $status, $msg ) = RT::Installer->SaveConfig;

Modified: rt/branches/3.8-TESTING/share/html/Prefs/Other.html
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Prefs/Other.html	(original)
+++ rt/branches/3.8-TESTING/share/html/Prefs/Other.html	Mon Jun  9 17:24:04 2008
@@ -70,7 +70,7 @@
 
 % if ( RT->Config->Get('GnuPG')->{'Enable'} ) {
 <&|/Widgets/TitleBox, title => loc( 'Cryptography' ) &>
-Preferred key: <& /Elements/GnuPG/SelectKeyForEncryption, EmailAddress => $UserObj->EmailAddress, Default => $UserObj->PreferredKey &>
+<&|/l&>Preferred key</&>: <& /Elements/GnuPG/SelectKeyForEncryption, EmailAddress => $UserObj->EmailAddress, Default => $UserObj->PreferredKey &>
 </&>
 % }
 

Modified: rt/branches/3.8-TESTING/share/html/REST/1.0/dhandler
==============================================================================
--- rt/branches/3.8-TESTING/share/html/REST/1.0/dhandler	(original)
+++ rt/branches/3.8-TESTING/share/html/REST/1.0/dhandler	Mon Jun  9 17:24:04 2008
@@ -63,7 +63,7 @@
 my $name   = qr{[\w.-]+};
 my $list   = '(?:(?:\d+-)?\d+,)*(?:\d+-)?\d+';
 my $label  = '[a-zA-Z0-9 at _.+-]+';
-my $field  = '[a-zA-Z](?:[a-zA-Z0-9_-]|\s+)*';
+my $field  = RT::Interface::REST->field_spec;
 my $labels = "(?:$label,)*$label";
 
 # We must handle requests such as the following:

Modified: rt/branches/3.8-TESTING/share/html/REST/1.0/search/ticket
==============================================================================
--- rt/branches/3.8-TESTING/share/html/REST/1.0/search/ticket	(original)
+++ rt/branches/3.8-TESTING/share/html/REST/1.0/search/ticket	Mon Jun  9 17:24:04 2008
@@ -54,12 +54,13 @@
 $fields => undef
 </%ARGS>
 <%INIT>
+use RT::Interface::REST;
 my $output = "";
 my $status = "200 Ok";
 my $tickets = new RT::Tickets $session{CurrentUser};
 
 # Parse and validate any field specifications.
-my $field  = '[a-zA-Z](?:[a-zA-Z0-9_-]|\s+)*';
+my $field  = RT::Interface::REST->field_spec;
 my (%fields, @fields);
 if ($fields) {
     $format ||= "l";

Modified: rt/branches/3.8-TESTING/share/html/Search/Elements/PickCFs
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Search/Elements/PickCFs	(original)
+++ rt/branches/3.8-TESTING/share/html/Search/Elements/PickCFs	Mon Jun  9 17:24:04 2008
@@ -58,6 +58,10 @@
         # XXX TODO: This ancient code dates from a former developer
         # we have no idea what it means or why cfqueues are so encoded.
         $id =~ s/^.'*(.*).'*$/$1/;
+
+        # unescape internal quotes
+        $id =~ s/(\\(.))/$2 eq "'" ? "'" : $1/eg;
+
         $queue->Load($id);
     }
     $CustomFields->LimitToQueue($queue->Id);

Modified: rt/branches/3.8-TESTING/share/html/Ticket/Elements/UpdateCc
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Ticket/Elements/UpdateCc	(original)
+++ rt/branches/3.8-TESTING/share/html/Ticket/Elements/UpdateCc	Mon Jun  9 17:24:04 2008
@@ -50,6 +50,7 @@
         <br />
 
 %foreach my $addr ( keys %txn_addresses) {
+%    next if ( grep {$addr eq $_} @req_addresses );
 <input 
 id="UpdateCc-<%$addr%>" 
 name="UpdateCc-<%$addr%>" 
@@ -60,6 +61,7 @@
 </td></tr>
 <tr><td align="right"><&|/l&>One-time Bcc</&>:</td><td><& /Elements/EmailInput, Name => 'UpdateBcc', Size => '60', Default => $ARGS{UpdateBcc} &><br />
 %foreach my $addr ( keys %txn_addresses) {
+%    next if ( grep {$addr eq $_} @req_addresses );
 <input
     id="UpdateBcc-<%$addr%>"
     name="UpdateBcc-<%$addr%>"
@@ -74,4 +76,5 @@
 </%args>
 <%init>
 my %txn_addresses = %{$TicketObj->TransactionAddresses};
+my @req_addresses = split /,/, $TicketObj->RequestorAddresses;
 </%init>

Modified: rt/branches/3.8-TESTING/share/html/User/Prefs.html
==============================================================================
--- rt/branches/3.8-TESTING/share/html/User/Prefs.html	(original)
+++ rt/branches/3.8-TESTING/share/html/User/Prefs.html	Mon Jun  9 17:24:04 2008
@@ -179,9 +179,7 @@
 <tr><td colspan="2" valign="top" class="boxcontainer">
 <&| /Widgets/TitleBox, title => loc('Secret authentication token'), id => "user-prefs-feeds" &>
 
-<p>All iCal feeds embed a secret token which authorizes you.  If the
-URL one of your iCal feeds got exposed to the outside world, you can
-get a new secret, <b>breaking all existing iCal feeds</b> below.</p>
+<&|/l&><p>All iCal feeds embed a secret token which authorizes you.  If the URL one of your iCal feeds got exposed to the outside world, you can get a new secret, <b>breaking all existing iCal feeds</b> below.</p></&>
 
 <& /Elements/Submit, Label => loc('Reset secret authentication token'), Name => "ResetAuthToken" &>
 </&>

Modified: rt/branches/3.8-TESTING/share/html/Widgets/TitleBoxStart
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Widgets/TitleBoxStart	(original)
+++ rt/branches/3.8-TESTING/share/html/Widgets/TitleBoxStart	Mon Jun  9 17:24:04 2008
@@ -48,10 +48,25 @@
 <div class="titlebox<% $class ? " $class " : '' %>" id="<% $id %>">
   <div class="titlebox-title<% $title_class ? " $title_class" : ''%>">
 % if ($hideable) {
-    <span class="widget"><a href="#" onclick="return rollup('<%$tid%>');" onfocus="this.blur(); return false;" title="Toggle visibility">X</a></span>
+    <span class="widget"><a href="#" 
+	onclick="return rollup('<%$tid%>');" 
+	onfocus="this.blur(); return false;" 
+	title="Toggle visibility">X</a>
+	</span>
 % }
-    <span class="left"><% $title_href ? qq[<a href="$title_href">] : '' | n %><% $title %><% $title_raw |n %><% $title_href ? "</a>" : '' |n%></span>
-    <span class="right<%($titleright_href || $titleright || $titleright_raw) ? '' : '-empty' %>"><% $titleright_href ? qq[<a href="$titleright_href">] : '' | n %><% $titleright  %><% $titleright_raw |n%> <% $titleright_href ? "</a>" : '' |n%></span>
+    <span class="left">
+      	<% $title_href ? qq[<a href="$title_href">] : '' | n %>
+      	<% $title %>
+     	<% $title_raw |n %>
+      	<% $title_href ? "</a>" : '' |n%>
+    </span>
+    <span class="right
+     	<%($titleright_href || $titleright || $titleright_raw) ? '' : '-empty' %>">
+	<% $titleright_href ? qq[<a href="$titleright_href">] : '' | n %>
+	<% $titleright  %>
+	<% $titleright_raw |n%> 
+	<% $titleright_href ? "</a>" : '' |n%>
+    </span>
   </div>
   <div class="titlebox-content <% $bodyclass %>" id="<% $tid %>">
 
@@ -75,8 +90,10 @@
 #
 my $page = $m->request_comp->path;
 
+my $title_b64 = MIME::Base64::encode_base64(Encode::encode_utf8($title), '');
+
 my $tid  = "TitleBox--$page--" .
-            join '--', ($class, $bodyclass, $title, $id);
+            join '--', ($class, $bodyclass, $title_b64, $id);
 
 $tid =~ s{[/\s'\\]+}{_}g;
 


More information about the Rt-commit mailing list