? Success

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
28 Nov 2014

This is just to give an idea of how I would solve the issue described in #4687

To test this test submitting a content article in the frontend. On Mysql this should keep working, on postgres is used to fail it should now pass.

In the backend both postgres and mysql should keep working as expected

Update

Recently in staging the frontend validation was fixed by another PR. I disagree with the way it was fixed in #4687 for many reasons. So as part of this PR I am also reverting that one (note as it is not in any release yet it is totally fine to do this)

avatar wilsonge wilsonge - open - 28 Nov 2014
avatar jissues-bot jissues-bot - change - 28 Nov 2014
Labels Added: ?
avatar wilsonge wilsonge - change - 28 Nov 2014
The description was changed
avatar alikon
alikon - comment - 28 Nov 2014

@test success
now you are able to create article from frontend as per #3087 on postgresql

avatar wilsonge wilsonge - change - 2 Dec 2014
The description was changed
avatar wilsonge wilsonge - change - 2 Dec 2014
Category Postgresql
avatar brianteeman
brianteeman - comment - 2 Dec 2014

After applying the patch in mysql and submitting an article on the front end I got the following error BUT the article was saved

screen shot 2014-12-02 at 17 26 48



This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5230.

avatar brianteeman brianteeman - test_item - 2 Dec 2014 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 2 Dec 2014

In postgres pressing save just gives me an empty content form again with no error or success message but the article is created

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5230.

avatar wilsonge
wilsonge - comment - 3 Dec 2014

OK from a code review/very quick physical test the error in mysql is unrelated to this and occurs when you create a duplicate alias for an item. The string exists in the backend as Alias already existed so a number was added at the end. You can re-edit the article to customise the alias which would tie in with this fact. I can replicate this without my patch applied so I would call it "unrelated" (although I guess it should be fixed for 3.4). Gonna try and figure out why no messages at all in postgres now :)

avatar wilsonge
wilsonge - comment - 3 Dec 2014

OK Really nice spot Brian! Turns out I'd missed although the article was being created the assets were not because the rules field was blank. This is fixed now. Please test again :)

avatar nikosdion
nikosdion - comment - 3 Dec 2014

@test Successful. I was able to submit and edit an article from the front-end. I also confirmed that doing the same from the back-end still works. I even tried playing around with ACLs in the front-end and back-end. Postgres 9.3.5.2 (through Postgress.app) on Mac OS X Yosemite. Great job, George!

avatar brianteeman
brianteeman - comment - 3 Dec 2014

@test all good now - setting RTC

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5230.

avatar brianteeman brianteeman - test_item - 3 Dec 2014 - Tested successfully
avatar brianteeman brianteeman - alter_testresult - 3 Dec 2014 - nikosdion: Tested successfully
avatar brianteeman brianteeman - change - 3 Dec 2014
Status Pending Ready to Commit
avatar brianteeman brianteeman - change - 3 Dec 2014
Labels Added: ?
avatar rdeutz rdeutz - close - 16 Jan 2015
avatar zero-24 zero-24 - close - 16 Jan 2015
avatar rdeutz rdeutz - change - 16 Jan 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-01-16 16:51:06
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment