? ? Success

User tests: Successful: Unsuccessful:

avatar SniperSister
SniperSister
22 Mar 2014

Fixed the remaining, non BC- or template-related code style issues

avatar SniperSister SniperSister - open - 22 Mar 2014
avatar wilsonge
wilsonge - comment - 23 Mar 2014

We need to change back the other $_variableName instances as well for the same b/c reasons as above :/

avatar Bakual
Bakual - comment - 23 Mar 2014

We need to change back the other $_variableName instances as well for the same b/c reasons as above :/

No, they are fine since those are private properties. They can't be accessed by extending classes :smile:

avatar Bakual
Bakual - comment - 24 May 2014

@SniperSister Can you rebase this PR and solve the conflicts?

avatar SniperSister
SniperSister - comment - 25 May 2014

To which branch should I rebase the PR?

avatar Bakual
Bakual - comment - 25 May 2014

Staging please.

avatar SniperSister
SniperSister - comment - 26 May 2014

More or less done - but I think i screwed up the commit history :(

avatar Bakual
Bakual - comment - 26 May 2014

The history doesn't concern me much. I can deal with that. But it contains also changed files which are not part of your original PR. Not sure what happend there.

avatar wilsonge
wilsonge - comment - 26 May 2014

What's really weird is some of those changes marked have already been made in staging (the removing of the beeze message file etc.)

avatar SniperSister SniperSister - change - 26 May 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-05-26 12:53:19
avatar SniperSister SniperSister - close - 26 May 2014
avatar SniperSister SniperSister - close - 26 May 2014
avatar SniperSister SniperSister - head_ref_deleted - 26 May 2014
avatar SniperSister SniperSister - change - 26 May 2014
Status Closed New
avatar SniperSister SniperSister - reopen - 26 May 2014
avatar SniperSister SniperSister - reopen - 26 May 2014
avatar SniperSister SniperSister - change - 26 May 2014
Status New Closed
Closed_Date 2014-05-26 12:53:19 2014-05-26 12:55:30
avatar SniperSister SniperSister - close - 26 May 2014
avatar SniperSister
SniperSister - comment - 26 May 2014

Okay, I'll try to start a new PR

avatar SniperSister SniperSister - close - 26 May 2014
avatar SniperSister SniperSister - head_ref_restored - 26 May 2014

Add a Comment

Login with GitHub to post a comment