Feeling Lucky
?
[#4062] - [#34037] multilanguage: cannot update an associated article from the frontend
- Closed
- 4 Aug 2014
- Medium
- Build: staging
- # 4062
- Diff
- infograf768:multilang_editarticle
- Foreign ID 34037
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
See the tracker for instructions
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=34037&start=0
jissues-bot
- | Labels |
Added:
?
?
|
||
| Title |
|
||||||
| Title |
|
||||||
| Title |
|
||||||
| Title |
|
||||||
infograf768
- comment
- 4 Aug 2014
Yes, it is.
If I change line 374
if ($id && $app->isSite() && $assoc)
to
if ($this->getState('article.id') && $app->isSite() && $assoc)
then the error is gone.
remains to decide what is the best solution.
What do you suggest?
Bakual
- comment
- 4 Aug 2014
I wonder why there is a difference between $id and $this->getState('article.id') in edits. I'd have to dig through where this state is set.
However I'd say if that PR broke it by changing from $this->getState('article.id') to $id and reverting that line fixes it. Then go with that approach. It was only a sidechange in that PR anyway.
infograf768
- comment
- 4 Aug 2014
Good test on jc + mine. Merging.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-08-04 08:45:55 |
Is this a side effect of #3944 ?