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.

Opened 11 years ago

Closed 11 years ago

#949 closed bug (fixed)

jSession: cannot use NOW() in dao storage because sqlite doesn't support it

Reported by: Julien Owned by: Julien
Priority: normal Milestone: Jelix 1.1.4
Component: jelix:core:jSession Version: 1.1.3
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

so we use jDateTime on the php side.

Attachments (1)

949-jSession-dao-without-now-function.diff (2.2 KB) - added by Julien 11 years ago.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by Julien

comment:1 follow-up: Changed 11 years ago by laurentj

  • review changed from review? to review-

Using jDatetime is overkill in this case I think. date('Y-m-d H:i:s') is enough, no ? :-) We should keep the jDatetime use only for convertion I think.

For the short term, ok for this patch. But it would be better if we could have a mecanism in jDao (and driver for dao), to convert automatically this kind of little SQL instruction to the corresponding one in the target database... (just an idea, I didn't investigate).

comment:2 in reply to: ↑ 1 Changed 11 years ago by Julien

  • Status changed from new to assigned

Replying to laurentj:

Using jDatetime is overkill in this case I think. date('Y-m-d H:i:s') is enough, no ? :-)

true ;) Will commit it that way.

We should keep the jDatetime use only for convertion I think. For the short term, ok for this patch. But it would be better if we could have a mecanism in jDao (and driver for dao), to convert automatically this kind of little SQL instruction to the corresponding one in the target database... (just an idea, I didn't investigate).

this would be really nice I think.

comment:3 Changed 11 years ago by Julien

  • Resolution set to fixed
  • Status changed from assigned to closed

committed in the trunk and 1.1.x, r1448

Note: See TracTickets for help on using tickets.