Show
Ignore:
Timestamp:
05/10/09 01:34:13 (4 years ago)
Author:
wiseoldowl
Message:

If redirect selected make sure redirect field is not empty or whitespace only

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • contributed_modules/modules/routepermissions/module.xml

    r7718 r7719  
    22  <rawname>routepermissions</rawname> 
    33  <name>Outbound Route Permissions</name> 
    4   <version>0.3.2.1</version> 
     4  <version>0.3.2.2</version> 
    55  <candisable>yes</candisable> 
    66  <canuninstall>yes</canuninstall> 
    77  <changelog> 
     8    *0.3.2.2* If redirect selected make sure redirect field is not empty or whitespace only 
    89    *0.3.2.1* Minor changes to instructions on main page 
    910    *0.3.2.0* Bug fixes in Ticket #3655 and Ticket #3669 
  • contributed_modules/modules/routepermissions/page.routepermissions.php

    r7718 r7719  
    5353      if (!strncmp($r, "redirect_", 8)) { 
    5454        $route=substr($r,9); 
    55         print "<h4>Route $route set to DENY for supplied range using redirect prefix</h4>\n"; 
    56         rp_redir($route, $_REQUEST["range_$route"], $_REQUEST["rp-redir_$route"]); 
     55        $redir=trim($_REQUEST["rp-redir_$route"]); 
     56        // Make sure redirect field is not empty or whitespace only - could have better sanity checking 
     57        if (strlen($redir)) { 
     58          print "<h4>Route $route set to DENY for supplied range using redirect prefix $redir</h4>\n"; 
     59          rp_redir($route, $_REQUEST["range_$route"], $redir); 
     60        } else { 
     61          print "<h4>Redirect selected but redirect prefix missing for route $route - no action taken</h4>\n"; 
     62        } 
    5763      } 
    5864      if ($r == 'update_dest') {