avatar waader
waader
16 Oct 2014

My testing environment:

  • Centos 6.5
  • php 5.4.16
  • PostgreSQL 9.3.5
  • Joomla 3.3.7-dev with com_patchtester 2.0.0 beta

When I try to apply path #3292 I get the following error:
Error
Database query failed (error # %s): %s SQL=INSERT INTO "zbjnd_patchtester_tests" ("pull_id","data","patched_by","applied","applied_version") VALUES (3292,'[{"action":"modified","index":"18d5a53..10bd3b9 100644","old":"modules/mod_articles_archive/helper.php","new":"modules/mod_articles_archive/helper.php","body":"\n\n

Applying the same patch using the same testing environment with mysql went find.

avatar waader waader - open - 16 Oct 2014
avatar mbabker
mbabker - comment - 16 Oct 2014

You had the patch tester installed on the PostgreSQL environment, correct? (Just trying to confirm and running on not as much sleep as normal right now)

avatar waader
waader - comment - 16 Oct 2014

Correct! I have postgresql 9.3.5 running under CentOS 6.5

Von: Michael Babker [mailto:notifications@github.com]
Gesendet: Donnerstag, 16. Oktober 2014 23:03
An: joomla-extensions/patchtester
Cc: waader
Betreff: Re: [patchtester] Appling patch #3292 using a J3.3.7-dev with a postgresql db fails (#41)

You had the patch tester installed on the PostgreSQL environment, correct? (Just trying to confirm and running on not as much sleep as normal right now)


Reply to this email directly or view it on GitHub #41 (comment) . https://github.com/notifications/beacon/9270310__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyOTExMjU5NiwiZGF0YSI6eyJpZCI6NDYwNDEzNDJ9fQ==--ca4a8335fedb6816cd167785d0126a5bc0b312e1.gif

avatar alikon
alikon - comment - 17 Oct 2014

The issue arise when you try to apply a patch
The problem is the insert cause lacks the fields rating which is declared not null
So for a quick fix you can change that field from not null to not null with default 0
Or
Correct the the SQL insert adding the rating field

avatar mbabker
mbabker - comment - 17 Oct 2014

From what I can see, the rating isn't even used (feature idea that wasn't implemented before?).

avatar mbabker mbabker - close - 17 Oct 2014
avatar mbabker
mbabker - comment - 17 Oct 2014

Fixed in master.

Add a Comment

Login with GitHub to post a comment