l193,194,195,196class\xoopsform\form.php

• Relator: mikhail
• Dono: minahito
• Tipo: XOOPSCube2.1 Bugs
• Estado: Fechado
• Componente: (Nenhum)
• Marcos: (Nenhum)
• Prioridade: 6
• Gravidade: 5 - Medium
• Resolução: Accepted
• Arquivo: Nenhum

Details Responder

I receive a email with a possible problem with the
lines 193,194,195,196 of form.php file:

if (!$this->_elements[$i]->isContainer()) {
$ret[] =& $this->_elements[$i];
} else {
$elements =&
$this->_elements[$i]->getElements(true);


Her sollution:


if(!is_object($this->_elements[$i])){
$ret[] =& $this->_elements[$i];
}elseif (!$this->_elements[$i]->isContainer()) {
$ret[] =& $this->_elements[$i];
} else {
$elements =& $this->_elements[$i]->getElements(true);


Her explanation in portuguese: O problema é que a
função "insertBreak" aceita strings, mas a getElements
que estava antes, ao fazer um loop por todos os
elementos do formulário, esperava que todos fossem
objetos. A alteração verifica antes se o elemento é
mesmo um objeto, para que possa usar as funções dele
sem dar problemas.

Ticket History (3/5 Histories)

2006-11-04 05:29 Updated by: mikhail

Comentário Responder

Logged In: YES
user_id=24765

Original:
http://br.groups.yahoo.com/group/xoopscube/message/3397

2006-11-04 05:33 Updated by: mikhail

• Prioridade Update from 5 - Medium to 6
• (Grupo change on Tracker)

2006-11-05 14:18 Updated by: minahito

• Dono Update from (Nenhum) to minahito
• (Category change on Tracker)
• Resolução Update from Nenhum to Accepted

2006-11-05 15:19 Updated by: minahito

Comentário Responder

Logged In: YES
user_id=8353

Thank you for your report & patch! I've just fixed.

2006-11-05 15:19 Updated by: minahito

• Ticket Close date is changed to 2006-11-05 15:19
• Estado Update from Aberto to Fechado