Changeset 1084 for trunk/preferences.cgi


Ignore:
Timestamp:
Jun 16, 2007, 10:33:16 PM (15 years ago)
Author:
Dagfinn Ilmari Mannsåker
Message:

Preferences cleanup

  • Separate preference defaulting from cookie parsing.
  • Remove duplicate preference defaulting from the cgi.
  • Use the prefs template variable in the template.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/preferences.cgi

    r752 r1084  
    2323
    2424sub set_preferences {
    25     my $username     = $cgi->param("username")                   || "";
    26     my $gc_link      = $cgi->param("include_geocache_link")      || 0;
    27     my $pre_above    = $cgi->param("preview_above_edit_box")     || 0;
    28     my $latlong_trad = $cgi->param("latlong_traditional")        || 0;
    29     my $omit_hlplnks = $cgi->param("omit_help_links")            || 0;
    30     my $rc_minor_eds = $cgi->param("show_minor_edits_in_rc")     || 0;
    31     my $edit_type    = $cgi->param("default_edit_type")          || "normal";
    32     my $expires      = $cgi->param("cookie_expires")             || "month";
    33     my $track_rc     = $cgi->param("track_recent_changes_views") || 0;
    34     my $gmaps        = $cgi->param("display_google_maps")        || 0;
     25    my %prefs = OpenGuides::CGI->get_prefs_from_hash( $cgi->Vars );
    3526    my $prefs_cookie = OpenGuides::CGI->make_prefs_cookie(
    3627        config => $config,
    37         username => $username,
    38         include_geocache_link  => $gc_link,
    39         preview_above_edit_box => $pre_above,
    40         latlong_traditional    => $latlong_trad,
    41         omit_help_links        => $omit_hlplnks,
    42         show_minor_edits_in_rc => $rc_minor_eds,
    43         default_edit_type      => $edit_type,
    44         cookie_expires         => $expires,
    45         track_recent_changes_views => $track_rc,
    46         display_google_maps    => $gmaps
     28        %prefs,
    4729    );
    4830    my @cookies = ( $prefs_cookie );
     
    6345        vars     => {
    6446                      not_editable               => 1,
    65                       username                   => $username,
    66                       include_geocache_link      => $gc_link,
    67                       preview_above_edit_box     => $pre_above,
    68                       latlong_traditional        => $latlong_trad,
    69                       omit_help_links            => $omit_hlplnks,
    70                       show_minor_edits_in_rc     => $rc_minor_eds,
    71                       default_edit_type          => $edit_type,
    72                       cookie_expires             => $expires,
    73                       track_recent_changes_views => $track_rc,
    74                       display_google_maps        => $gmaps
     47                      %prefs,
    7548                    }
    7649    );
     
    7851
    7952sub show_form {
    80     # Get defaults for form fields from cookies.
    81     my %prefs = OpenGuides::CGI->get_prefs_from_cookie( config => $config );
    82 
    8353    print OpenGuides::Template->output(
    8454        wiki     => $wiki,
    8555        config   => $config,
    8656        template => "preferences.tt",
    87         vars     => { %prefs,
     57        vars     => {
    8858                      not_editable => 1,
    8959                      show_form    => 1
Note: See TracChangeset for help on using the changeset viewer.