Skip to content

CLEANING up JArchiveZip#6515

Merged
Kubik-Rubik merged 1 commit intojoomla:stagingfrom
Mathewlenning:CLEANING-JArchiveZip
May 8, 2016
Merged

CLEANING up JArchiveZip#6515
Kubik-Rubik merged 1 commit intojoomla:stagingfrom
Mathewlenning:CLEANING-JArchiveZip

Conversation

@Mathewlenning
Copy link
Copy Markdown
Contributor

Same treatment as the others.

  1. Isolating JError into a temporary method.
  2. Refactored JArchiveZip::_getFileData to remove multiple elseIf clauses.

isolating JError into a temporary method.
@Kubik-Rubik Kubik-Rubik added this to the Joomla! 3.5.0 milestone Mar 20, 2015
@wilsonge wilsonge removed this from the Joomla! 3.5.0 milestone Nov 6, 2015
@Kubik-Rubik
Copy link
Copy Markdown
Member

Thank you @Mathewlenning!

@Kubik-Rubik Kubik-Rubik merged commit de64732 into joomla:staging May 8, 2016
roland-d added a commit to roland-d/joomla-cms that referenced this pull request May 8, 2016
…leanup-installer-plugins

* 'staging' of https://github.com/joomla/joomla-cms:
  Smart Search: utf8_strpos: Offset must be an integer (joomla#10303)
  Removed an redundant else clause from JFeedParser::processElement (joomla#7961)
  CLEANING-JDataSet (joomla#7947)
  CLEANING-JAdapter (joomla#6679)
  Same  treatment as JArchiveBzip2 and JArchiveGzip (joomla#6515)
  Cleaning up JArchiveBzip2 (joomla#6495)
  Removed the nested conditional switch construct (joomla#6338)
  add edit tooltip to modules (joomla#10295)
  New installer plugins: remove the dot in the plugin name and other language review (joomla#10299)
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.

4 participants