developer.jelix.org is not used any more and exists only for
history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
Opened 11 years ago
Closed 11 years ago
#1139 closed bug (fixed)
jFormsControlDate.class.php et jFormsControlDatetime
Reported by: | Zeffyr | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | Jelix 1.1.7 |
Component: | jelix:forms | Version: | 1.1.6 |
Severity: | normal | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Documentation needed: | no | |
Hosting Provider: | Php version: |
Description
lib/jelix/forms/controls/jFormsControlDate.class.php
function setValueFromRequest($request) { $value = $request->getParam($this->ref,''); $value = $value['year'].'-'.$value['month'].'-'.$value['day']; if($value == '--') $value = ''; $this->setData($value); }
il manque un test sur $value qui peut ne pas être un tableau patch proposé:
function setValueFromRequest($request) { $value = $request->getParam($this->ref,''); if(is_array($value)) $value = $value['year'].'-'.$value['month'].'-'.$value['day']; if($value == '--') $value = ''; $this->setData($value); }
Idem pour lib/jelix/forms/controls/jFormsControlDatetime.class.php
function setValueFromRequest($request) { $value = $request->getParam($this->ref,''); if($value['year'] === '' && $value['month'] === '' && $value['day'] === '' && $value['hour'] === '' && $value['minutes'] === '' && (!$this->enableSeconds || $value['seconds'] === '')) $this->setData(''); else{ if($value['seconds']==='') $value['seconds'] = '00'; $this->setData($value['year'].'-'.$value['month'].'-'.$value['day'].' '.$value['hour'].':'.$value['minutes'].':'.$value['seconds']); } }
par
function setValueFromRequest($request) { $value = $request->getParam($this->ref,''); if(!is_array($value)) $this->setData(''); elseif($value['year'] === '' && $value['month'] === '' && $value['day'] === '' && $value['hour'] === '' && $value['minutes'] === '' && (!$this->enableSeconds || $value['seconds'] === '')) $this->setData(''); else{ if($value['seconds']==='') $value['seconds'] = '00'; $this->setData($value['year'].'-'.$value['month'].'-'.$value['day'].' '.$value['hour'].':'.$value['minutes'].':'.$value['seconds']); } }
Change History (2)
comment:1 Changed 11 years ago by laurentj
- Component changed from jelix to jelix:forms
- Milestone set to Jelix 1.2
comment:2 Changed 11 years ago by laurentj
- Milestone changed from Jelix 1.2 to Jelix 1.1.7
- Resolution set to fixed
- review set to review+
- Status changed from new to closed
- Version changed from trunk to 1.1.6
patch quand même appliqué. merci.
Note: See
TracTickets for help on using
tickets.
Dans quel cas ce n'est pas un tableau ?