Skip to content

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Oct 28, 2016

This file was still lying around used in a few places that could (and should) use \OCP\IL10N etc.

  • Convert OC_L10N usages
  • Cleanup tests
  • Remove OC_L10N
  • Remove OC_L10N tests

CC: @LukasReschke @nickvergessen @MorrisJobke

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added the 3. to review Waiting for reviews label Oct 28, 2016
@rullzer rullzer added this to the Nextcloud 11.0 milestone Oct 28, 2016
@mention-bot
Copy link

@rullzer, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @nickvergessen and @bartv2 to be potential reviewers.

@MorrisJobke
Copy link
Member

I will review this and while doing this also fix all the apps in our repo.

@MorrisJobke
Copy link
Member

We should put this in the release notes ;)

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke
Copy link
Member

Tested and works so far 👍

@codecov-io
Copy link

Current coverage is 57.15% (diff: 50.00%)

Merging #1948 into master will decrease coverage by 0.07%

@@             master      #1948   diff @@
==========================================
  Files          1079       1078     -1   
  Lines         61585      61551    -34   
  Methods        6886       6899    +13   
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          35242      35179    -63   
- Misses        26343      26372    +29   
  Partials          0          0          

Sunburst

Diff Coverage File Path
0% settings/personal.php
0% lib/private/Setup/AbstractDatabase.php
0% lib/private/TemplateLayout.php
0% apps/files/lib/Helper.php
•••••••••• 100% ...ps/federatedfilesharing/templates/settings-admin.php
•••••••••• 100% apps/files/templates/recentlist.php
•••••••••• 100% lib/private/legacy/util.php
•••••••••• 100% lib/private/legacy/l10n/string.php
•••••••••• 100% apps/files/templates/index.php
•••••••••• 100% core/templates/login.php

Review all 23 files changed

Powered by Codecov. Last update f2cae3c...a78f6e9

@nickvergessen
Copy link
Member

👍

@LukasReschke LukasReschke merged commit e48fa1c into master Oct 29, 2016
@LukasReschke LukasReschke deleted the move_away_lagacy_oc_l10n branch October 29, 2016 07:39
@MorrisJobke
Copy link
Member

@rullzer Doc update? Release notes ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants