Skip to content

Conversation

@frankmayer
Copy link
Contributor

@frankmayer frankmayer commented Dec 13, 2016

Summary of Changes

  • Merged unset() calls

This PR is part of a set to try to separate some of the changes done in some of my previous batch PR's for site/components, which are still on hold (#12290, #12292, #12293, #12294).

Testing Instructions

None, should not change behavior

Documentation Changes Required

None.

@alikon
Copy link
Contributor

alikon commented Dec 13, 2016

I have tested this item ✅ successfully on 4e941e2

on code review


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

@andrepereiradasilva
Copy link
Contributor

I have tested this item ✅ successfully on 4e941e2

code review


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

@jeckodevelopment
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 13, 2016
@jeckodevelopment jeckodevelopment added this to the Joomla 3.7.0 milestone Dec 13, 2016
@rdeutz rdeutz merged commit d827bba into joomla:staging Dec 14, 2016
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Dec 14, 2016
@frankmayer frankmayer deleted the merging-unset-calls-in-site-components branch December 14, 2016 21:08
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.

6 participants