#6403: ifelse cannot be reached
------------------------+---------------------------------------------------
    Reporter:  remi     |         Owner:           
        Type:  Bug      |        Status:  reopened 
    Priority:  Medium   |     Milestone:  1.2.x.x  
   Component:  General  |       Version:  1.2 Final
    Severity:  Normal   |    Resolution:           
    Keywords:           |   Php_version:  n/a      
Cake_version:           |  
------------------------+---------------------------------------------------
Changes (by juan_basso):

  * status:  closed => reopened
  * resolution:  needmoreinfo =>

Comment:

 He is saying that the logic of if's are wrong. Is not entering elseif
 here.

 For exemple, if isset($names[$key]) is true, we have:
 {{{
 if (true) { // Enter here
 ...
 } elseif (true && empty($names[$key]) && empty($out[$names[$key]])) { //
 Not here
 ...
 }}}

 But, if isset($names[$key]) is false, we have:
 {{{
 if (false) { // Not here
 ...
 } elseif (false && empty($names[$key]) && empty($out[$names[$key]])) { //
 Here all is AND comparation, not enter again...
 ...
 }}}

 This should be anything as:
 {{{
 if (isset($names[$key])) {
   if (empty($names[$key]) && empty($out[$names[$key]])) {
     break;
   }
   $out[$names[$key]] = $_this->stripEscape($found);
 } else {
   ...
 }}}

-- 
Ticket URL: <https://trac.cakephp.org/ticket/6403#comment:3>
CakePHP : The Rapid Development Framework for PHP <https://trac.cakephp.org/>
Cake is a rapid development framework for PHP which uses commonly known design 
patterns like ActiveRecord, Association Data Mapping, Front Controller and MVC. 
Our primary goal is to provide a structured framework that enables PHP users at 
all levels to rapidly develop robust web applications, without any loss to 
flexibility.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"tickets cakephp" group.
To post to this group, send email to tickets-cakephp@googlegroups.com
To unsubscribe from this group, send email to 
tickets-cakephp+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/tickets-cakephp?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to