[#13758] - [com_associations] moved "checked out" check to a helper function
- Fixed in Code Base
- 27 Jan 2017
- Medium
- Build: staging
- # 13758
- Diff
- rdeutz:move_checkout_to_helper
User tests: Successful: Unsuccessful:
Summary of Changes
This PR moves the check if an item is checked out to the helper function and used this function when creating links to items
Testing Instructions
- install latest staging with 2 or more languages
- create a 2nd user
- open 2 browsers and log in with both users
- browser A: open Multilingual Associations extension
- browser A: chose article and a language
- browser B: open an article (edit)
You will see that you can access an checked out article, this end with an error message but that's not nice.
Apply the patch
Now you can see that articles have the locked sign and also the associated items don't have a link to open it.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration |
I didn't changed the article view I only worked on the associations view. Seems to me that is as it was before
sorry i was confused by
items don't have a link to open it.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13758.
Not sure it works as should. Maybe need clearer test instructions.
User2 using BrowserB is a manager with access to com_associations.
After patch, here is the page in browser B showing the article which is checked out by another user in browserA.
It does show the lock (greyed as Manager has no com_checkin permission) but the article is still clickable.
Now I click on the article title and I get this.
As you can see, not only I can click on an item which is locked, but also I get weird results in the side by side and when closing and although the user/manager has no com_checkin permissions, the article is unlocked and the associated article (FR in this case) is locked.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13758.
Ok got it. This here moved the check if something is checked out to a helper function.
The question at hand is, what is the right behaviour?
a) If we say the article at the left is something like the source than it doesn't makes any sense to edit this or on the associated articles (because some is changing the source).
b) If this can be source or target than it is much more complicated.
What we can do is to disable the link on the article in the associations view, when the article is checked out
| Labels |
Added:
?
|
||
I made a fix following the a) behaviour
OK now: a locked item as well as its associations button (when present) do not open the side by side.
There are other issues to solve but I can mark this as tested OK for what it already does.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13758.
hope i've not missed some test case this turn ;)
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13758.
| Status | Pending | ⇒ | Ready to Commit |
so we can set RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13758.
jeckodevelopment
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-01-27 19:56:17 |
| Closed_By | ⇒ | jeckodevelopment | |
| Labels |
Added:
?
|
||
i've logged in in browser A with admin1
i've logged in in browser B with admin2
after applying the patch on browser B i see the locked icon on article IT and it associated article GB
but both have a link to open is this wanted behaviuor ?