Skip to content
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

Remove usage of deprecated JArrayHelper - com_menus #10675

Closed
wants to merge 2 commits into from

Conversation

izharaazmi
Copy link
Contributor

@izharaazmi izharaazmi commented May 30, 2016

Remove usage of deprecated JArrayHelper from com_menus

Testing Instructions

Test:

  • Set default menu item
  • Publish/Unpublish
  • Batch copy/move etc.

Also review code, if possible.

@andrepereiradasilva
Copy link
Contributor

I have tested this item ✅ successfully on 23320f0

Works as described.
Code review fine.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10675.

@ghost
Copy link

ghost commented Jun 11, 2016

I have tested this item ✅ successfully on 23320f0


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10675.

@brianteeman
Copy link
Contributor

Rtc


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10675.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 11, 2016
@wilsonge wilsonge added this to the Joomla 3.7.0 milestone Jun 15, 2016
@wilsonge
Copy link
Contributor

Merged with 14ff580 - thanks!

@wilsonge wilsonge closed this Jul 30, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 30, 2016
@wilsonge wilsonge removed the RTC This Pull Request is Ready To Commit label Jul 30, 2016
@izharaazmi izharaazmi deleted the jarrayhelper-com-menus branch August 23, 2016 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants