Feeling Lucky
?
[#4842] - Regression: Fixing tags on PostgreSQL
- Closed
- 19 Oct 2014
- Medium
- Build: staging
- # 4842
- Diff
- Bakual:FixTagsAttemp53276
- Success The Travis CI build passed Details
Referenced as
Related to:
# 4114
User tests: Successful: Unsuccessful:
Issue
According to @wilsonge PR 4114 broke tags on PostgreSQL (see #4114 (comment)).
Solution
This PR adds the column a.lft to the select as suggested by @alikon.
Votes
# of Users Experiencing Issue
1/1
Average Importance Score
5.00
jissues-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Postgresql |
wilsonge
- comment
- 18 Oct 2014
@brianteeman would you mind testing this please?
brianteeman
- comment
- 18 Oct 2014
Not from 30,000 fleet sorry
On 18 Oct 2014 19:54, "George Wilson" notifications@github.com wrote:
@brianteeman https://github.com/brianteeman would you mind testing this
please?—
Reply to this email directly or view it on GitHub
#4842 (comment).
ketchupmonki
- comment
- 19 Oct 2014
Tested successfully.
| Status | Pending | ⇒ | Ready to Commit |
wilsonge
- comment
- 19 Oct 2014
RTC - thanks guys!
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-19 18:25:47 |
To test this check you can save a tag in the joomla backend (specifically the parent field shows the expected values). Note that on saving a tag in postgres you will still get a SQL error after applying this patch on save - you'll need to apply this patch for an independent issue to save the tag #4815
test: works correctly after patch