Default values on a PG table: type cast appended

[qgib]

Author Name: Paolo Cavallini (@pcav)
Original Redmine Issue: 13857
Affected QGIS version: 2.8.3
Redmine category:data_provider/postgis
Assignee: Jürgen Fischer

---

If the table has a default value, this is used in QGIS, but quotes and a type cast is added to it, e.g.:

'test123'::character varying

instead of:

test123

---

Related issue(s): #20565 (duplicates), #21782 (relates)
Redmine related issue(s): 12391, 13755

---

[qgib]

Author Name: Jürgen Fischer (@jef-n)

---

Defaults values are kept verbatim (not only for @'test123'::character varying@, but also @nextval('foo_id_seq'::regclass)@). What's the problem with it?

---

• status_id was changed from Open to Feedback

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

e.g. if I insert a path to a photo, and set the field widget to photo, the image itself fails to be displayed because of the additional text added.

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

If the table does not have a PK, and only one field, apparently the cast is not added.

[qgib]

Author Name: Jürgen Fischer (@jef-n)

---

Paolo Cavallini wrote:

If the table does not have a PK, and only one field, apparently the cast is not added.

Can't reproduce that with:

qgis=> create table issue13857(file varchar default '/tmp/foo.png');
CREATE TABLE
qgis=> \\d issue13857
                       Table "public.issue13857"
 Column |       Type        |                 Modifiers                 
--------+-------------------+-------------------------------------------
 file   | character varying | default '/tmp/foo.png'::character varying

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

OK, so last think must have been a local mistake. Sorry about the noise.

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

• resolution was changed from to invalid
• status_id was changed from Feedback to Closed

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

Giovanni, please don't close valid issues. Thanks.

---

• status_id was changed from Closed to Reopened

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

Paolo Cavallini wrote:

Giovanni, please don't close valid issues. Thanks.

didn't you stated it was a 'local mistake'?

---

• resolution was changed from invalid to

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

• status_id was changed from Reopened to Feedback

[qgib]

Author Name: Paolo Cavallini (@pcav)

---

This refers to the PK issue, not to the main one, which is still valid.

---

• status_id was changed from Feedback to Open

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

I cannot confirm the specific issue described here, a default defined as

'test123'::character varying

returns as expected

test123

after adding new records and saving.

On the other hand a default defined as

('now'::text)::date

(in a date column of course), works if adding a record in pgadmin (and other backends) but in QGIS does not work and it returns always NULL.

QGIS 2.8.4, pgsql 9.3 and postgis 2.1

---

• status_id was changed from Open to Feedback

[qgib]

Author Name: Jürgen Fischer (@jef-n)

---

Giovanni Manghi wrote:

('now'::text)::date

(in a date column of course), works if adding a record in pgadmin (and other backhands) but in QGIS does not work and it returns always NULL
QGIS 2.8.4, pgsql 9.3 and postgis 2.1

works fine for me in master too (PostgreSQL 9.4)

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

Jürgen Fischer wrote:

Giovanni Manghi wrote:

('now'::text)::date

(in a date column of course), works if adding a record in pgadmin (and other backhands) but in QGIS does not work and it returns always NULL
QGIS 2.8.4, pgsql 9.3 and postgis 2.1

works fine for me in master too (PostgreSQL 9.4)

confirmed also in another platform that on master

('now'::text)::date

works ok but not on 2.8.5, while

'test123'::character varying

works as expected on both.

[qgib]

Author Name: Jürgen Fischer (@jef-n)

---

Fixed in changeset "4c3cf64f01af32511ef5fc56b0b13aa705c00dd6".

---

• status_id was changed from Feedback to Closed