User tests: Successful: Unsuccessful:
There is already two uses of try/catch in 3.3-dev (https://github.com/joomla/joomla-cms/blob/3.3-dev/components/com_content/models/article.php#L319 and https://github.com/joomla/joomla-cms/blob/3.3-dev/components/com_content/models/article.php#L346 ) But you can see the instance I'm replacing here https://github.com/joomla/joomla-cms/blob/3.3-dev/components/com_content/models/article.php#L299
Which should similarly use the try/catch. So not it's not resolved :)
Thanks for the follow up George!
I looked at the code before, and I do see the lack of try / catch
I applied the patch, reviewed the code again, and now see the try / catch
I enabled article voting, voted on an article, and it worked successfully
Looks good to go on my end.
You may blame the J!Tracker Application for transmitting this comment.
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-05-30 22:30:06 |
Hi wilsonge,
I'm looking at the storeVote() function within components/com_content/models/article.php. In joomla 3.3 dev, I am seeing a try / catch after the execute, such as this:
Does the issue appear to be resolved, or, am I not understanding your question?
You may blame the J!Tracker Application for transmitting this comment.