diff options
author | INOPIAE <inopiae@cacert.org> | 2013-07-09 23:27:55 +0200 |
---|---|---|
committer | INOPIAE <inopiae@cacert.org> | 2013-07-09 23:27:55 +0200 |
commit | 5f3c22a2e6edbeedd735b917dc069008d0aa77a3 (patch) | |
tree | 9bbef82699e07da2ac9805f06f809b5aa34417b0 /pages/account/13.php | |
parent | 54f4f30e058fdea779a09ca3a2063b948a4ef67f (diff) | |
download | cacert-devel-5f3c22a2e6edbeedd735b917dc069008d0aa77a3.tar.gz cacert-devel-5f3c22a2e6edbeedd735b917dc069008d0aa77a3.tar.xz cacert-devel-5f3c22a2e6edbeedd735b917dc069008d0aa77a3.zip |
bug 663: changed Unconditional Variable Assignment instead of if-condition for $showdetails
Diffstat (limited to 'pages/account/13.php')
-rw-r--r-- | pages/account/13.php | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/pages/account/13.php b/pages/account/13.php index 6ba1ff9..0abfd94 100644 --- a/pages/account/13.php +++ b/pages/account/13.php @@ -23,7 +23,7 @@ $year = intval(substr($user['dob'], 0, 4)); $month = intval(substr($user['dob'], 5, 2)); $day = intval(substr($user['dob'], 8, 2)); - $showdetails=0; if(array_key_exists('showdetails',$_REQUEST)) $showdetails=intval($_REQUEST['showdetails']); + $showdetails = array_key_exists('showdetails', $_REQUEST) && !!intval($_REQUEST['showdetails']); if($showdetails == 1){ $body = sprintf(_("Hi %s,"),$user['fname'])."\n\n"; $body .= _("You receive this automatic mail since you yourself or someone ". @@ -118,12 +118,12 @@ </tr> <tr> <td class="DataTD"><?=_("Date of Birth")?><br> - (<?=_("dd/mm/yyyy")?>)</td> + (<?=_("dd/mm/yyyy")?>)</td> <td class="DataTD"><?=$day?> <?=ucwords(recode("utf-8..html", strftime("%B", mktime(0,0,0,$month,1,1))))?> <?=$year?></td> </tr> <? } ?> <tr> - <td colspan="2" class="title"><a href="account.php?id=13&showdetails=<?=!$showdetails?>"><?=_("View secret question and answers and OTP phrases")?></a></td> + <td colspan="2" class="title"><a href="account.php?id=13&showdetails=<?=!$showdetails?>"><?=_("View secret question & answers and OTP phrases")?></a></td> </tr> <? if($showdetails == 1){ ?> <tr> |