-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[com_contact] Make ACL core.edit.own work (PR for 11466) #11503
Conversation
I have tested this item ✅ successfully on 3282bfd This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11503. |
Note: The PR works great. |
I have tested this item ✅ successfully on 3282bfd This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11503. |
I have tested this item 🔴 unsuccessfully on 3282bfd [07-Aug-2016 07:10:36 UTC] PHP Notice: Undefined variable: record in ROOT/administrator/components/com_contact/controllers/contact.php on line 82 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11503. |
sorry guys. corrected that. please test. |
no more notice after last update |
I have tested this item ✅ successfully on a1fb945 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11503. |
I have tested this item ✅ successfully on a1fb945 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11503. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11503. |
@wilsonge is this ok to merge or does it bring back a certain issue |
this looks good to me @wilsonge, have set the milestone, your turn to press the merge button |
I'm just going to hold on this until i can test some stuff ;) |
ok @wilsonge, go ahead, more tests are alwys good. Just informing that are (at least) 4 PR similiar to this one:
So if you find something all need to be changed. |
Oh didn't realise we'd merged the other two. Merging this then. We can go back |
* re-arrayhelper-min: (2467 commits) Minimize JArrayHelper methods `toInteger`, `pivot`, `arrayUnique` by using Joomla\Utilities\ArrayHelper internally. Leaving (reverting from joomla#7782) other four methods as is for b/c reasons as mentioned in joomla#8455. remove platform include (joomla#11615) [GitHub Templates] Make headings bigger (joomla#11607) [com_contact] Make ACL core.edit.own work (PR for 11466) (joomla#11503) Small review on docs & code structure in JModelLegacy library classes (joomla#11057) Obviously, this should be an array. (joomla#11610) Don't manually import JPlatform anymore (joomla#10841) Parse preprocess rules from component routers (joomla#8986) Add the correct exception after 11593 merge (was waiting for that merrge) (joomla#11606) Add missing clean line after joomla#9277 (joomla#11605) Deprecate the _PROFILER global var (joomla#10845) Spelling errors (joomla#11604) Moved travis javascript bash file to build/travis like joomla#11600 (joomla#11603) Regression: Fix edit check in backend articles manager, always denying edit after soft deny (joomla#11511) [com_plugins] User not allowed to core.manage? Use 403 php custom exception (instead of a 404 JError) (joomla#11593) [com_newsfeeds] Make ACL core.edit.own work (PR for 11466) (joomla#11502) $result-variable-undefined-given-default-value (joomla#9277) com_banners use exceptions. and not allowed is a 403 (joomla#11418) Frontend & plugins using the autoloader (joomla#10882) New version of PR 6788 (JText::_() Optimizations) (joomla#11235) ...
* fix core.edit.own * correct variable
* fix core.edit.own * correct variable
Pull Request for Issue #11466 (com_contact part).
Summary of Changes
The ACL core.edit.own is not working in com_contact.
This PR makes it work.
Testing Instructions
Also use the two users to do a general test with the com_contact edit permissions to confirm all is fine.
Documentation Changes Required
None.
Notes
This problem was discovered in GsoC 2016 Multilingual project.
This is very similiar with #11502.