Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#1381 tiki-1.8-pgsql.sql bad "sessions" declaration

v1.8.1
open
7
2004-06-24
2004-03-16
No

There's a mistake in db/tiki-1.8-pgsql.sql about the
declaration of the "sessions" table (line 257)

The columns names should be in lower case and not
uppercase as they are.

This bug make an error message appear about "Session
Replace" at the end of each generated page.

Making the column names lowercase fixes the problem.
Maybe there should be a little sql upgrade script in
next version for that ?

---
DROP TABLE "sessions";

CREATE TABLE "sessions"(
"sesskey" char(32) NOT NULL,
"expiry" bigint NOT NULL,
"expireref" varchar(64),
"data" text NOT NULL,
PRIMARY KEY ("sesskey")

) ;
---

thanks for the great work by the way ;)

Discussion

  • Logged In: YES
    user_id=738765

    redflo can you check why we ended up with this?

     
    • assigned_to: nobody --> redflo
    • summary: Bad "sessions" table creation in PostgreSQL init script --> Bad
     
  • Logged In: YES
    user_id=183182

    Thanks for the hint. I'm busy these days, but i think, the
    problem is adodb's session handler. It does not follow
    tiki's database abstraction conventions, and it does not add
    quotes to the tables/columns in the queries. So we have the
    problem, that different database systems convert different:
    Oracle converts to upper, mysql to lower ....
    possible Solutions:
    - find out if adodb's session handler has a "quote" option
    or
    - write our own session handler
    or
    - add two versions of the session table: one "sessions" with
    lower case columns and one "SESSIONS" with uc cols in the
    creation script.

    Flo

     
  • teedog
    teedog
    2004-06-24

    Logged In: YES
    user_id=807810

    Has this been resolved yet?

     
  • teedog
    teedog
    2004-06-24

    • priority: 5 --> 7
    • summary: Bad --> tiki-1.8-pgsql.sql bad "sessions" declaration
     
  • Marc Laporte
    Marc Laporte
    2008-04-08

    Logged In: YES
    user_id=484395
    Originator: NO

    bump