Skip to content

Cleanups, fixes and a bit of optimizations for site/components batch #2#12291

Merged
rdeutz merged 2 commits intojoomla:stagingfrom
frankmayer:site-com_contact
Oct 18, 2016
Merged

Cleanups, fixes and a bit of optimizations for site/components batch #2#12291
rdeutz merged 2 commits intojoomla:stagingfrom
frankmayer:site-com_contact

Conversation

@frankmayer
Copy link
Copy Markdown
Contributor

@frankmayer frankmayer commented Oct 3, 2016

  • com_contact

Note: This is a single commit bundling all types of changes, since PR #12261 which had detailed commits, was rejected as a whole

- com_contact

Note: This is a single commit bundling all types of changes, since PR #12261 which had detailed commits, was rejected as a whole
$temp = new Registry;
$temp->loadString($item->params);
$item->params = clone($this->params);
$item->params = clone$this->params;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add a space here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops... ;)

@andrepereiradasilva
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 2372fcb

on code review


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

@wilsonge wilsonge added this to the Joomla! 3.6.4 milestone Oct 3, 2016
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Oct 3, 2016

RTC on review for 3.6.4


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 3, 2016
@brianteeman
Copy link
Copy Markdown
Contributor

Milestone changed to 3.7 as it is now not planned to have a 3.6.4 release

@rdeutz rdeutz merged commit 80d5b5e into joomla:staging Oct 18, 2016
@brianteeman brianteeman removed the RTC This Pull Request is Ready To Commit label Oct 18, 2016
@frankmayer frankmayer deleted the site-com_contact branch October 18, 2016 21:28
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