? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
18 Oct 2014

From our testing sprint this week, we've taken some initial steps to modernize some of our architecture and focus on replacing static methods as able (this will be blogged about more thoroughly in the coming weeks). To do so without backward compatibility breaks, we've introduced several wrapper classes which are object based and enable developers to utilize these to work with what will be the object based API in a later major release.

This PR demonstrates how the wrappers can be used internally by using the newly created wrappers for the Filesystem package within its classes. In conjunction with #4717 this demonstrates how we can begin modernizing the code in phased steps without introducing larger B/C breaks between major releases by providing a compatibility layer in our current releases.

Testing Instructions

  • Apply the patch
  • Perform actions in the user interface that interact with the filesystem (the media manager and extension manager are great places to test from)
  • Ensure filesystem management still works
avatar mbabker mbabker - open - 18 Oct 2014
avatar jissues-bot jissues-bot - change - 18 Oct 2014
Labels Added: ?
avatar brianteeman brianteeman - change - 18 Oct 2014
Category Libraries
avatar mbabker
mbabker - comment - 21 Jan 2015

:shrug:

avatar mbabker mbabker - change - 21 Jan 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-01-21 12:22:31
avatar mbabker mbabker - close - 21 Jan 2015
avatar brianteeman
brianteeman - comment - 21 Jan 2015

Was this closed because of?

avatar mbabker
mbabker - comment - 21 Jan 2015

Lack of interest, to be able to delete the 3.4-dev branch

avatar javigomez
javigomez - comment - 21 Jan 2015

But it can be recreated against staging, isn't it?

avatar Kubik-Rubik
Kubik-Rubik - comment - 21 Jan 2015

@mbabker Can you do a new PR against staging, please?

avatar mbabker
mbabker - comment - 21 Jan 2015

See #5849

Add a Comment

Login with GitHub to post a comment