[Rt-commit] r13083 - in rt/branches/3.8-TESTING: . bin etc lib/RT/I18N lib/RT/Interface sbin share/html/Admin/Elements 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:23:26 EDT 2008


Author: jesse
Date: Mon Jun  9 17:23:22 2008
New Revision: 13083

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/Installer.pm
   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/Admin/Elements/SelectStage
   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/HeaderJavascript
   rt/branches/3.8-TESTING/share/html/Elements/TicketList
   rt/branches/3.8-TESTING/share/html/Install/DatabaseDetails.html
   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:23:22 2008
@@ -94,7 +94,7 @@
 # (XXX: Ask Autrijus how i18n changes these definitions.)
 
 my $name   = '[\w.-]+';
-my $field  = '(?i:[a-z][a-z0-9_-]*|C(?:ustom)?F(?:ield)?-(?:[a-z0-9_ :()/-]|\s)+)';
+my $field  = '(?:[a-zA-Z](?:[a-zA-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") if defined($type);
+        @objects = ("$type/new");
     }
     #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:23:22 2008
@@ -1,3 +1,7 @@
+#
+# WARNING: NEVER EDIT RT_Config.pm. Instead, copy any sections you want to change to RT_SiteConfig.pm
+# and edit them there.
+#
 
 package RT;
 
@@ -11,579 +15,327 @@
 
 =cut
 
-=head1 WARNING
+# {{{ Base Configuration
 
-NEVER EDIT RT_Config.pm.
+# $rtname is the string that RT will look for in mail messages to
+# figure out what ticket a new piece of mail belongs to
 
-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
+# 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
 
 Set($rtname , "example.com");
 
 
-=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 );>
+# 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 );
 
-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 );
 
-
-
-=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
+# 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.
 
 Set($Organization , "example.com");
 
-=item C<$MinimumPasswordLength>
-
-C<$MinimumPasswordLength> defines the minimum length for user
-passwords. Setting it to 0 disables this check.
-
-=cut
-
+# $MinimumPasswordLength defines the minimum length for user
+# passwords. Setting it to 0 disables this check
 Set($MinimumPasswordLength , "5");
 
-=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
-
+# $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.
 Set($Timezone , 'US/Eastern');
 
-=back
-
-=head1 Database Configuration
-
-=over 4
+# }}}
 
-=item C<$DatabaseType>
+# {{{ Database Configuration
 
-Database driver being used; case matters.
-
-Valid types are "mysql", "Oracle" and "Pg"
-
-=cut
+# Database driver being used; case matters.  Valid types are "mysql",
+# "Oracle" and "Pg"
 
 Set($DatabaseType , '@DB_TYPE@');
 
-=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
-
+# The domain name of your database server
+# If you're running mysql and it's on localhost,
+# leave it blank for enhanced performance
 Set($DatabaseHost   , '@DB_HOST@');
 Set($DatabaseRTHost , '@DB_RT_HOST@');
 
-=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
-
+# The port that your database server is running on.  Ignored unless it's
+# a positive integer. It's usually safe to leave this blank
 Set($DatabasePort , '@DB_PORT@');
 
-=item C<$DatabaseUser>
-
-The name of the database user (inside the database)
-
-=cut
-
+#The name of the database user (inside the database)
 Set($DatabaseUser , '@DB_RT_USER@');
 
-=item C<$DatabasePassword>
-
-Password the C<$DatabaseUser> should use to access the database
-
-=cut
-
+# Password the DatabaseUser should use to access the database
 Set($DatabasePassword , '@DB_RT_PASS@');
 
-=item C<$DatabaseName>
-
-The name of the RT's database on your database server
-
-=cut
-
+# The name of the RT's database on your database server
 Set($DatabaseName , '@DB_DATABASE@');
 
-=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
-
+# If you're using Postgres and have compiled in SSL support,
+# set DatabaseRequireSSL to 1 to turn on SSL communication
 Set($DatabaseRequireSSL , undef);
 
-=back
-
-=head1 Incoming Mail Gateway Configuration
-
-=over 4
+# }}}
 
-=item C<$OwnerEmail>
+# {{{ Incoming mail gateway configuration
 
-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
+# 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.
 
 Set($OwnerEmail , 'root');
 
-=item C<$LoopsToRTOwner>
-
-If C<$LoopsToRTOwner> is defined, RT will send mail that it believes
-might be a loop to C<$OwnerEmail>
-
-=cut
+# If $LoopsToRTOwner is defined, RT will send mail that it believes
+# might be a loop to $OwnerEmail
 
 Set($LoopsToRTOwner , 1);
 
-=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
+# 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
 
 Set($StoreLoops , undef);
 
-=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
+# $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)
 
 Set($MaxAttachmentSize , 10000000);
 
-=item C<$TruncateLongAttachments>
-
-C<$TruncateLongAttachments>: if this is set to a non-undef value,
-RT will truncate attachments longer than C<$MaxAttachmentSize>.
-
-=cut
+# $TruncateLongAttachments: if this is set to a non-undef value,
+# RT will truncate attachments longer than MaxAttachmentSize.
 
 Set($TruncateLongAttachments , undef);
 
-=item C<$DropLongAttachments>
-
-C<$DropLongAttachments>: if this is set to a non-undef value,
-RT will silently drop attachments longer than C<MaxAttachmentSize>.
-
-=cut
+# $DropLongAttachments: if this is set to a non-undef value,
+# RT will silently drop attachments longer than MaxAttachmentSize.
 
 Set($DropLongAttachments , undef);
 
-=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
+# 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.
 
 Set($ParseNewMessageForTicketCcs , undef);
 
-=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
+# RTAddressRegexp is used to make sure RT doesn't add itself as a ticket CC if
+# the setting above is enabled.
 
 Set($RTAddressRegexp , '^rt\@example.com$');
 
-=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
+# 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
 
 #Set($CanonicalizeEmailAddressMatch , '@subdomain\.example\.com$');
 #Set($CanonicalizeEmailAddressReplace , '@example.com');
 
-=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 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.
 
 Set($SenderMustExistInExternalDatabase , undef);
 
-=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>
+# @MailPlugins is a list of auth plugins for L<RT::Interface::Email>
+# to use; see L<rt-mailgate>
 
-The default "extract remote tracking tags" scrip settings; these
-detect when your RT is talking to another RT, and adjusts the
-subject accordingly.
+# $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
 
+# 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
+# }}}
 
-=over 4
+# {{{ Outgoing mail configuration
 
-=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
+# $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'
 
 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");
 
-
-=item C<$SendmailBounceArguments>
-
-C<$SendmailBounceArguments> defines what flags to pass to C<$Sendmail>
-assuming RT needs to send an error (ie. bounce).
-
-=cut
-
+# $SendmailBounceArguments defines what flags to pass to $Sendmail
+# assuming RT needs to send an error (ie. bounce).
 Set($SendmailBounceArguments , '-f "<>"');
-
-=item C<$SendmailPath>
-
-If you selected 'sendmailpipe' above, you MUST specify the path to
-your sendmail binary in C<$SendmailPath>.
-
-=cut
-
+# If you selected 'sendmailpipe' above, you MUST specify the path to
+# your sendmail binary in $SendmailPath.
 Set($SendmailPath , "/usr/sbin/sendmail");
 
+# }}}
 
-=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
+# {{{ SMTP configuration
+# These options only take effect if $MailCommand is 'smtp'
 
+# $SMTPServer should be set to the hostname of the SMTP server to use
 Set($SMTPServer, undef);
-
-=item C<$SMTPFrom>
-
-C<$SMTPFrom> should be set to the 'From' address to use, if not the
-email's 'From'
-
-=cut
-
+# $SMTPFrom should be set to the 'From' address to use, if not the
+# email's 'From'
 Set($SMTPFrom, undef);
-
-=item C<$SMTPDebug> 
-
-C<$SMTPDebug> should be set to true to debug SMTP mail sending
-
-=cut
-
+# $SMTPDebug should be set to true to debug SMTP mail sending
 Set($SMTPDebug, 0);
+# }}}
 
-=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
-
+# {{{ Other mailer configuration
+# @MailParams defines a list of options passed to $MailCommand if it
+# is not 'sendmailpipe', 'sendmail', or 'smtp'
 Set(@MailParams, ());
 
-=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.
+# RT is designed such that any mail which already has a ticket-id associated
+# with it will get to the right place automatically.
 
-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
+# $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.
 
 Set($CorrespondAddress , '');
 
 Set($CommentAddress , '');
 
-=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.
+# By default, RT sets the outgoing mail's "From:" header to
+# "SenderName via RT".  Setting UseFriendlyFromLine to 0 disables it.
 
-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.
+Set($UseFriendlyFromLine , 1);
 
-=cut
-
-Set($UseFriendlyToLine, 0);
+# sprintf() format of the friendly 'From:' header; its arguments
+# are SenderName and SenderEmailAddress.
+Set($FriendlyFromLineFormat , "\"%s via RT\" <%s>");
 
-=item C<$FriendlyToLineFormat>
+# RT can optionally set a "Friendly" 'To:' header when sending messages to
+# Ccs or AdminCcs (rather than having a blank 'To:' header.
 
-C<sprintf()> format of the friendly 'From:' header; its arguments
-are WatcherType and TicketId.
+# 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.
 
-=cut
+Set($UseFriendlyToLine , 0);
 
+# sprintf() format of the friendly 'From:' header; its arguments
+# are WatcherType and TicketId.
 Set($FriendlyToLineFormat, "\"%s of ". RT->Config->Get('rtname') ." Ticket #%s\":;");
 
-=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
+# 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
 
 Set($NotifyActor, 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
+# By default, RT records each message it sends out to its own internal database.
+# To change this behavior, set $RecordOutgoingEmail to 0 
 
 Set($RecordOutgoingEmail, 1);
 
-=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
-
+# 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.
 # Set($VERPPrefix, 'rt-');
 # Set($VERPDomain, $RT::Organization);
 
 
-=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
+# 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.
 
 Set($ForwardFromUser, 0);
 
-=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
-
+# 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
 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).
+# }}}
 
-=over 4
+# {{{ 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).
 
-=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,
 );
 
-=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
+# 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.
 
 Set(%GnuPGOptions,
     homedir => '@RT_VAR_PATH@/data/gpg',
@@ -598,426 +350,284 @@
 #    'auto-key-retrieve' => undef,
 );
 
+# }}}
 
-=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
 
+# 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');
 
-=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
+# 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.
 
 Set($LogToFile      , undef);
 Set($LogDir, '@RT_LOG_PATH@');
 Set($LogToFileNamed , "rt.log");    #log to rt.log
 
-=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
+# 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.)
 
 Set(@LogToSyslogConf, ());
 
-=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
-
+# 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.
 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:
+# }}}
 
-  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
+# {{{ Web interface configuration
 
+# 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');
 
-=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.
+# Define the directory name to be used for images in rt web
+# documents.
 
-C<$WebPath> requires a leading / but no trailing /.
-
-In most cases, you should leave C<$WebPath> set to '' (an empty value).
-
-=cut
+# 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).
 
 Set($WebPath, "");
 
-=item C<$WebPort>
-
-If we're running as a superuser, run on port 80
-Otherwise, pick a high port for this user.
-
-=cut
+# If we're running as a superuser, run on port 80
+# Otherwise, pick a high port for this user.
 
 Set($WebPort, 80);# + ($< * 7274) % 32766 + ($< && 1024));
 
-=item C<$WebDomain>
-
-you know what domain name is, right? ;)
-
-=cut
-
+# you know what domain name is, right? ;)
 Set( $WebDomain, 'localhost' );
 
-=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
+# This is the Scheme, server and port for constructing urls to webrt
+# $WebBaseURL doesn't need a trailing /
 
 Set($WebBaseURL, 'http://' . RT->Config->Get('WebDomain') . ':' . RT->Config->Get('WebPort'));
 
 Set($WebURL, RT->Config->Get('WebBaseURL') . RT->Config->Get('WebPath') . "/");
 
-=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
+# $WebImagesURL points to the base URL where RT can find its images.
 
 Set($WebImagesURL, RT->Config->Get('WebPath') . "/NoAuth/images/");
 
-=item C<$LogoURL>
-
-C<$LogoURL> points to the URL of the RT logo displayed in the web UI
-
-=cut
+# $LogoURL points to the URL of the RT logo displayed in the web UI
 
 Set($LogoURL, $Config->Get('WebImagesURL') . "bplogo.gif");
 
-=item C<$WebNoAuthRegex>
-
-What portion of RT's URLspace should not require authentication.
-
-=cut
-
+# WebNoAuthRegex - What portion of RT's URLspace should not require
+# authentication.
 Set($WebNoAuthRegex, qr{^ (?:/+NoAuth/ | /+REST/\d+\.\d+/NoAuth/) }x );
 
-=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
-
+# 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
 Set($MessageBoxWidth, 72);
 Set($MessageBoxHeight, 15);
 
-=item C<$MessageBoxWrap>
-
-Default wrapping: "HARD"  (choices "SOFT", "HARD")
-
-=cut
-
+# Default wrapping: "HARD"  (choices "SOFT", "HARD")
 Set($MessageBoxWrap, "HARD");
 
-=item C<$MessageBoxRichText>
-
-Should "rich text" editing be enabled? This option lets your users send html email messages from the web interface.
-
-=cut
-
+# Should "rich text" editing be enabled? This option lets your users send html email messages from the web interface.
 Set($MessageBoxRichText, 1);
 
-=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
-
+# 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.
 Set($WikiImplicitLinks, 0);
 
-=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
-
+# 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.
 Set($RedistributeAutoGeneratedMessages, 'privileged');
 
-=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
-
+# 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.
 Set($PreferRichText, undef);
 
-=item C<$WebExternalAuth>
-
-If C<$WebExternalAuth> is defined, RT will defer to the environment's
-REMOTE_USER variable.
-
-=cut
+# If $WebExternalAuth is defined, RT will defer to the environment's
+# REMOTE_USER variable.
 
-Set($WebExternalAuth, undef);
+Set($WebExternalAuth , undef);
 
-=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
+# If $WebFallbackToInternalAuth is undefined, the user is allowed a chance
+# of fallback to the login screen, even if REMOTE_USER failed.
 
 Set($WebFallbackToInternalAuth , undef);
 
-=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
+# $WebExternalGecos means to match 'gecos' field as the user identity);
+# useful with mod_auth_pwcheck and IIS Integrated Windows logon.
 
 Set($WebExternalGecos , undef);
 
-=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
+# $WebExternalAuto will create users under the same name as REMOTE_USER
+# upon login, if it's missing in the Users table.
 
 Set($WebExternalAuto , undef);
 
-=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
+# 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
 
 Set($AutoCreate, undef);
 
-=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
+# $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.
 
 # Set($WebSessionClass , 'Apache::Session::File');
 
-=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
+# 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.
 
 Set($AutoLogoff, 0);
 
-=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
+# 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.
 
 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.
 
-=cut
+# 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.
 
 Set($WebFlushDbCacheEveryRequest, '1');
 
 
-=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
+# $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.
 
 Set($MaxInlineBody, 12000);
 
-=item C<$DefaultSummaryRows>
-
-C<$DefaultSummaryRows> is default number of rows displayed in for search
-results on the frontpage.
-
-=cut
+# $DefaultSummaryRows is default number of rows displayed in for search
+# results on the frontpage.
 
 Set($DefaultSummaryRows, 10);
 
-=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
+# 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.
 
 Set($OldestTransactionsFirst, '1');
 
-=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
+# 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
 
 Set($ShowTransactionImages, 1);
 
 
-=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
+# 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.
 
 Set($ShowUnreadMessageNotifications, 1);
 
 
-=item C<$HomepageComponents>
-
-C<$HomepageComponents> is an arrayref of allowed components on a user's
-customized homepage ("RT at a glance").
-
-=cut
+# $HomepageComponents is an arrayref of allowed components on a user's
+# customized homepage ("RT at a glance").
 
 Set($HomepageComponents, [qw(QuickCreate Quicksearch MyAdminQueues MySupportQueues MyReminders RefreshHomepage)]);
 
-=item C<@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:
-
-    use MasonX::Profiler; # available on CPAN
-    @MasonParameters = (preamble => 'my $p = MasonX::Profiler->new($m, $r);');
-
-=cut
+# @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);');
 
 Set(@MasonParameters, ());
 
-=item C<$DefaultSearchResultFormat>
+# 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);
 
-C<$DefaultSearchResultFormat> is the default format for RT search results
+# 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);
 
+# %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, ());
+# }}}
+
+# $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',
@@ -1033,339 +643,169 @@
    '<small>__LastUpdatedRelative__</small>',
    '<small>__TimeLeft__</small>'});
 
-
-=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
+# 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.
 
 Set($SuppressInlineTextFiles, undef);
 
-=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
+# If $DontSearchFileAttachments is set to a true value, then uploaded
+# files (attachments with file names) are not searched during full-content
+# ticket searches.
 
 Set($DontSearchFileAttachments, undef);
 
-=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
+# 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.
 
 Set($ChartFont, undef);
 
 
-=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
-
+# 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.
 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
 
-=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
+# 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.
 
 Set(@LexiconLanguages, qw(*));
 
-=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
+# An array that contains default encodings used to guess which charset
+# an attachment uses if not specified.  Must be recognized by
+# Encode::Guess.
 
 Set(@EmailInputEncodings, qw(utf-8 iso-8859-1 us-ascii));
 
-=item C<$EmailOutputEncoding>
-
-The charset for localized email.  Must be recognized by Encode.
-
-=cut
-
-Set($EmailOutputEncoding, 'utf-8');
-
-
-=back
-
-=head1 Date Handling Configuration
+# The charset for localized email.  Must be recognized by Encode.
 
-=over 4
+Set($EmailOutputEncoding , 'utf-8');
 
-=item C<$DateTimeFormat>
+# }}}
 
-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
+# {{{ RT Date Handling Options
 
+# 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
-
-=item C<$DateDayBeforeMonth>
-
-Set this to 1 if your local date convention looks like "dd/mm/yy"
-instead of "mm/dd/yy".
-
-=cut
+# Set this to 1 if your local date convention looks like "dd/mm/yy"
+# instead of "mm/dd/yy".
 
 Set($DateDayBeforeMonth , 1);
 
-=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
+# 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.
 
 Set($AmbiguousDayInPast, 0);
 Set($AmbiguousDayInFuture, 0);
 
-=back
-
-=head1 Miscellaneous Configuration
-
-=over 4
-
-=item C<@ActiveStatus>, C<@InactiveStatus>
+# }}}
 
-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.
+# {{{ Miscellaneous RT Settings
 
-=cut
+# 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.
 
 Set(@ActiveStatus, qw(new open stalled));
 Set(@InactiveStatus, qw(resolved rejected deleted));
 
-=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
-
+# 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.
 Set($StrictLinkACL, 1);
 
-=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 to 1 if the scrips preview on the ticket
+# reply page should include the content of the messages to be sent.
 Set($PreviewScripMessages, 0);
 
-=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 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.
 Set($UseTransactionBatch, 0);
 
-=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 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.
 Set(@CustomFieldValuesSources, ());
 
-=item C<$CanonicalizeRedirectURLs>
-
-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.
-
-=cut
+# 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.
 
 Set($CanonicalizeRedirectURLs, 0);
 
-=item C<$DisableGraphViz>
+# }}}
 
-=cut
-
-Set($DisableGraphViz, do { local $@; eval {require GraphViz; 1;} ? 0 : 1 } );
+# {{{ 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
+#
 
-=item C<$DisableGD>
+Set($DevelMode, '@RT_DEVEL_MODE@');
 
-=cut
+Set($DisableGraphViz, do { local $@; eval {require GraphViz; 1;} ? 0 : 1 } );
 
 Set($DisableGD, do { local $@; eval {require GD; 1;} ? 0 : 1 } );
 
+# }}}
 
-=item C<$EnableReminders>
-
-Hide links/portlets related to Reminders by setting this to 0
-
-=cut
+# Reminders
+# Hide links/portlets related to Reminders by setting this to 0
 
 Set($EnableReminders,1);
 
-=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
+# {{{ Deprecated options
 
-=cut
-
-Set($DevelMode, '@RT_DEVEL_MODE@');
-
-
-=back
-
-=head1 Deprecated Options
-
-=over 4
-
-=item C<$AlwaysUseBase64>
+# $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.
 
-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:23:22 2008
@@ -109,7 +109,7 @@
         },
     },
     DefaultSummaryRows => {
-        Section         => 'RT at a glance',    #loc
+        Section         => 'RT at a glance',
         Overridable     => 1,
         Widget          => '/Widgets/Form/Integer',
         WidgetArguments => {
@@ -187,7 +187,7 @@
         Overridable     => 1,
         Widget          => '/Widgets/Form/Select',
         WidgetArguments => {
-            Description => 'Default queue',    #loc
+            Description => 'Default queue',
             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',    #loc
+            Description => 'Email delivery',
             Values      => [
-                'Individual messages',    #loc
-                'Daily digest',           #loc
-                'Weekly digest',          #loc
-                'Suspended'               #loc
+                'Individual messages',
+                'Daily digest',
+                'Weekly digest',
+                'Suspended'
             ]
             }
             },

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:23:22 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\""
+msgstr "Sélectionnez les files à afficher dans la page \"RT en un coup d'œil\" page"
 
 #: 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/Installer.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Installer.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Installer.pm	Mon Jun  9 17:23:22 2008
@@ -150,7 +150,7 @@
     MaxAttachmentSize => {
         Widget          => '/Widgets/Form/Integer',
         WidgetArguments => {
-            Description => 'Max attachment size( in bytes )',             #loc
+            Description => 'Max attachment size',             #loc
         },
     },
     OwnerEmail => {
@@ -186,7 +186,7 @@
     SendmailPath => {
         Widget          => '/Widgets/Form/String',
         WidgetArguments => {
-            Description => 'where is sendmail command',                  #loc
+            Description => 'Sendmail path',                  #loc
         },
     },
     WebDomain => {

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:23:22 2008
@@ -63,7 +63,7 @@
     @EXPORT = qw(expand_list form_parse form_compose vpush vsplit);
 }
 
-sub field_spec { '(?i:[a-z][a-z0-9_-]*|C(?:ustom)?F(?:ield)?-(?:[a-z0-9_ :()/-]|\s)+)' }
+my $field = '(?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,7 +96,6 @@
     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:23:22 2008
@@ -114,7 +114,6 @@
 Set( \$DatabaseName , $dbname);
 Set( \$LogToSyslog , undef);
 Set( \$LogToScreen , "warning");
-Set( \$MailCommand, 'testfile');
 };
     print $config $args{'config'} if $args{'config'};
     print $config "\n1;\n";
@@ -345,27 +344,6 @@
     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:23:22 2008
@@ -121,7 +121,6 @@
     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/Admin/Elements/SelectStage
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Admin/Elements/SelectStage	(original)
+++ rt/branches/3.8-TESTING/share/html/Admin/Elements/SelectStage	Mon Jun  9 17:23:22 2008
@@ -47,26 +47,17 @@
 %# END BPS TAGGED BLOCK }}}
 <select name="<%$Name%>">
 % foreach my $stage (@stages) {
-
-%# allow [stage, display] in place of a scalar stage name
-% my ($value, $display) = ref($stage) ? @$stage : ($stage, $stage);
-
-<option value="<%$value%>"
-<% ($value eq $Default) && qq[ selected="selected"] |n %>
-><% loc($display) %>
+<option value="<%$stage%>"
+<% ($stage eq $Default) && qq[ selected="selected"] |n %>
+><% loc($stage) %>
 </option>
-
 % }
 <%INIT>
 if ( !defined $Default || $Default eq '') {
     $Default = 'TransactionCreate';
 }
 my @stages = 'TransactionCreate';
-
-push @stages, RT->Config->Get('UseTransactionBatch')
-            ? 'TransactionBatch'
-            : ['TransactionBatch', 'TransactionBatch (DISABLED)'];
-
+push @stages, 'TransactionBatch' if RT->Config->Get('UseTransactionBatch');
 push @stages, 'Disabled';
 </%INIT>
 <%ARGS>

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:23:22 2008
@@ -45,13 +45,61 @@
 %# those contributions and any derivatives thereof.
 %# 
 %# END BPS TAGGED BLOCK }}}
-<%INIT>
-if (!$Collection && $Class eq 'RT::Tickets') {
-    $Collection = RT::Tickets->new( $session{'CurrentUser'} );
-    $Collection->FromSQL($Query);
+<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,
+    );
 }
+</%PERL>
 
-my $TotalFound = $Collection->CountAll();
+</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;
 return '' if !$TotalFound && !$ShowEmpty;
 
 if ( @OrderBy ) {
@@ -66,8 +114,6 @@
 }
 
 $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 
@@ -94,84 +140,29 @@
     }
 }
 
-$Class ||= ref $Collection;
+my $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          => 1
+$Rows          => undef
 $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:23:22 2008
@@ -8,14 +8,11 @@
 </%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;
@@ -26,7 +23,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> });
@@ -39,14 +36,13 @@
 
 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/HeaderJavascript
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Elements/HeaderJavascript	(original)
+++ rt/branches/3.8-TESTING/share/html/Elements/HeaderJavascript	Mon Jun  9 17:23:22 2008
@@ -77,8 +77,8 @@
               || (textArea.getAttribute('className') == 'messagebox')) {
                 // Turn the original plain text content into HTML 
 
-%# Special case - when we re-edit a textarea, don't re-encode it
-% if (!$m->request_args->{'FCKeditorEncoded'}) {
+%# Special case - When continuing an edit session with attachment, keep the HTMLified content.
+% if (!$m->request_args->{'AddMoreAttach'} && !$m->request_args->{'UpdatePreview'}) {
                 textArea.value = textArea.value.replace(
                     /&/g, "&amp;"
                 ).replace(
@@ -90,14 +90,6 @@
                 );
 % }
 
-%# for this template..
-                var FCKeditorEncoded = document.createElement('input');
-                FCKeditorEncoded.setAttribute('type', 'hidden');
-                FCKeditorEncoded.setAttribute('name', 'FCKeditorEncoded');
-                FCKeditorEncoded.setAttribute('value', '1');
-                textArea.parentNode.appendChild(FCKeditorEncoded);
-
-%# for FCKeditor..
                 var typeField = document.createElement('input');
                 typeField.setAttribute('type', 'hidden');
                 typeField.setAttribute('name', textArea.name + 'Type');

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:23:22 2008
@@ -45,23 +45,164 @@
 %# 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>
-$m->comp(
-	 '/Elements/CollectionList',
-	 %ARGS,
-	 Class => 'RT::Tickets'
-	);
+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;
+    }
+}
+
 </%INIT>
 <%ARGS>
 $Collection => undef
+
 $Query => undef
 $Format => RT->Config->Get('DefaultSearchResultFormat')
 $DisplayFormat => undef
 $Order => undef
 $OrderBy => undef
 $Rows => 50
+
 $Page => 1
-$Title => loc('Ticket Search')
+$Title => 'Ticket Search'
 $BaseURL => undef
 $AllowSorting => undef
 $ShowNavigation => 1

Modified: rt/branches/3.8-TESTING/share/html/Install/DatabaseDetails.html
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Install/DatabaseDetails.html	(original)
+++ rt/branches/3.8-TESTING/share/html/Install/DatabaseDetails.html	Mon Jun  9 17:23:22 2008
@@ -51,14 +51,7 @@
 
 <form method="post">
 <& /Widgets/BulkEdit, Types => \@Types, Meta => $RT::Installer->{Meta},
-    CurrentValue => { %{RT::Installer->CurrentValues(@Types)},
-        DatabaseAdmin =>
-            RT::Installer->CurrentValue( 'DatabaseAdmin' ) ||
-            $RT::Installer->{InstallConfig}{DatabaseAdmin} ||
-            ( $db_type eq 'mysql' ? 'root'
-              : $db_type eq 'Pg' ? 'postgres'
-              : '' ) }
-                    &>
+    CurrentValue => RT::Installer->CurrentValues(@Types) &>
 <input type="hidden" name="Run" value="1">
 
 <& /Elements/Submit, Label => loc('Check Database Connectivity'), 
@@ -77,13 +70,11 @@
 
 my @Types = 'DatabaseName';
 
-my $db_type = $RT::Installer->{InstallConfig}{DatabaseType};
-
-unless ( $db_type eq 'SQLite' ) {
+unless ( $RT::Installer->{InstallConfig}{DatabaseType} eq 'SQLite' ) {
     push @Types, 'DatabaseHost', 'DatabasePort', 'DatabaseAdmin',
         'DatabaseAdminPassword', 'DatabaseUser', 'DatabasePassword';
 }
-if ( $db_type eq 'Pg' ) {
+if ( $RT::Installer->{InstallConfig}{DatabaseType} eq 'Pg' ) {
     push @Types, 'DatabaseRequireSSL';
 }
 

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:23:22 2008
@@ -49,14 +49,6 @@
 
 <& /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) &>
@@ -68,12 +60,8 @@
 </&>
 <%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/OwnerEmail CommentAddress CorrespondAddress/;
+my @Types = qw/MaxAttachmentSize OwnerEmail CommentAddress CorrespondAddress/;
 
 if ( $Run ) {
 
@@ -86,6 +74,12 @@
 '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:23:22 2008
@@ -49,6 +49,10 @@
 
 <& /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) &>
@@ -56,13 +60,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 Email Addresses') &>
+    BackLabel => loc('Back: Customize Emails') &>
 </form>
 </&>
 <%init>
 my @results;
 
-my @Types = qw/MaxAttachmentSize SendmailArguments SendmailBounceArguments SendmailPath/;
+my @Types = qw/SendmailArguments SendmailBounceArguments SendmailPath/;
 
 if ( $Run ) {
 
@@ -75,16 +79,6 @@
 'Install/Emails.html');
     }
 
-    for ( qw/MaxAttachmentSize/ ) {
-        if ( $ARGS{$_} && $ARGS{$_} !~ /^\d+$/ ) {
-            push @results, "Invalid $_: it should be a number";
-        }
-    }
-
-    unless ( -e $ARGS{SendmailPath} ) {
-        push @results, "$ARGS{SendmailPath} doesn't exist.";
-    }
-
     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:23:22 2008
@@ -70,7 +70,7 @@
 
 % if ( RT->Config->Get('GnuPG')->{'Enable'} ) {
 <&|/Widgets/TitleBox, title => loc( 'Cryptography' ) &>
-<&|/l&>Preferred key</&>: <& /Elements/GnuPG/SelectKeyForEncryption, EmailAddress => $UserObj->EmailAddress, Default => $UserObj->PreferredKey &>
+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:23:22 2008
@@ -63,7 +63,7 @@
 my $name   = qr{[\w.-]+};
 my $list   = '(?:(?:\d+-)?\d+,)*(?:\d+-)?\d+';
 my $label  = '[a-zA-Z0-9 at _.+-]+';
-my $field  = RT::Interface::REST->field_spec;
+my $field  = '[a-zA-Z](?:[a-zA-Z0-9_-]|\s+)*';
 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:23:22 2008
@@ -54,13 +54,12 @@
 $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  = RT::Interface::REST->field_spec;
+my $field  = '[a-zA-Z](?:[a-zA-Z0-9_-]|\s+)*';
 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:23:22 2008
@@ -58,10 +58,6 @@
         # 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:23:22 2008
@@ -50,7 +50,6 @@
         <br />
 
 %foreach my $addr ( keys %txn_addresses) {
-%    next if ( grep {$addr eq $_} @req_addresses );
 <input 
 id="UpdateCc-<%$addr%>" 
 name="UpdateCc-<%$addr%>" 
@@ -61,7 +60,6 @@
 </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%>"
@@ -76,5 +74,4 @@
 </%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:23:22 2008
@@ -179,7 +179,9 @@
 <tr><td colspan="2" valign="top" class="boxcontainer">
 <&| /Widgets/TitleBox, title => loc('Secret authentication token'), id => "user-prefs-feeds" &>
 
-<&|/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></&>
+<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:23:22 2008
@@ -48,25 +48,10 @@
 <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 %>">
 
@@ -90,10 +75,8 @@
 #
 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_b64, $id);
+            join '--', ($class, $bodyclass, $title, $id);
 
 $tid =~ s{[/\s'\\]+}{_}g;
 


More information about the Rt-commit mailing list