bug v1.0
avatar ndeet
ndeet
19 Nov 2013

Steps to reproduce the issue

Open any issue and set "I have experienced this issue" to yes and click on the vote button.

Expected result

Redirect to login or notice that only logged in members can vote.

Actual result

Error in the button: JTracker\Database\AbstractDatabaseTable::load can not bind.

System information (as much as possible)

Additional comments

avatar ndeet ndeet - open - 19 Nov 2013
avatar ndeet ndeet - open - 19 Nov 2013
avatar b2z
b2z - comment - 19 Nov 2013

Interesting. We closed it with #183, but seems that was fixed only for a logged in user.

avatar b2z
b2z - comment - 19 Nov 2013

Now it it gets more interesting :) Trying to do it as logged in and not logged in and have the same error as in #183: Trying to get property of non-object
You may blame the J!Tracker Application for transmitting this comment.

avatar ndeet
ndeet - comment - 19 Nov 2013

Did a search before posting but must have overseen #183. I have to mention that I used the tracker on http://issues.joomla.org/tracker/

Don't know if issues.joomla.org it working on current master or 1.0.0-beta - so maybe #183 already fixed it as you mentioned.
You may blame the J!Tracker Application for transmitting this comment.

avatar b2z
b2z - comment - 19 Nov 2013

Yeap, this is what I am thinking now also :) May be the latest code was not deployed. We should wait for Michael's reply :)
You may blame the J!Tracker Application for transmitting this comment.

avatar mbabker
mbabker - comment - 20 Nov 2013

Should have been fixed. My unscientific mechanism of tracking where the server is at says it's on 8652b03 for the moment, where I supposedly fixed this issue. Apparently I didn't get it on the first try though.

avatar ndeet
ndeet - comment - 20 Nov 2013

Should it work now? I just tried it as anonymous user again and now get (on issues.joomla.org/tracker using firefox)
Trying to get property of non-object

You may blame the J!Tracker Application for transmitting this comment.

avatar elkuku
elkuku - comment - 20 Nov 2013

Might I also confirm this ? ... Additionally you get different errors on the different projects - logged in or not.
I think a place to start would be here because what is used is the issue_number not the id, but I believe there is more... Or are we going to use some sort of hash tags, as suggested ?

avatar b2z
b2z - comment - 20 Nov 2013

Yeah seems like you are right. We need to change it in the template file then ;) I will look into it tomorrow.

avatar b2z
b2z - comment - 21 Nov 2013

Fixed, please deploy :) Did not close with the commit cause there was incorrect issue number in comment :blush:

avatar mbabker
mbabker - comment - 22 Nov 2013

Deployed latest, still getting the property of non-object message :-(

avatar b2z
b2z - comment - 22 Nov 2013

Interesting, on existing votes I can vote (for example - here), but new one does not count :) So I will dig again today, where is the problem :D

BTW you have not deployed issue.index.twig ;)

avatar b2z b2z - reference | - 22 Nov 13
avatar b2z
b2z - comment - 22 Nov 2013

I hope now it is ok :+1: I have tested on my site and voting works well.

avatar mbabker
mbabker - comment - 29 Nov 2013

This is deployed live, so it should be gone from the server now.

avatar ndeet
ndeet - comment - 29 Nov 2013

Thanks it works now!

One improvement for the future would be to allow only vote one time. If I now login/logout I can make limitless votes as anonymous and logged in user. But this is another issue.
You may blame the J!Tracker Application for transmitting this comment.

avatar b2z b2z - close - 30 Nov 2013
avatar b2z b2z - reference | - 2 Dec 13

Add a Comment

Login with GitHub to post a comment