Opened on 11/06/2014 at 01:05:40 PM

Closed on 11/24/2014 at 05:10:14 PM

Last modified on 02/23/2015 at 01:12:08 PM

#1540 closed defect (fixed)

Utility method for loading text is at the wrong location

Reported by: rjeschke Assignee: rjeschke
Priority: P2 Milestone: Adblock-Plus-for-Android-1.3
Module: Adblock-Plus-for-Android Keywords:
Cc: Blocked By:
Blocking: Platform: Android
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/5662276015095808

Description

There's a method called appendRawTextFile which is currently declared in class AdblockPlus but only used in AboutDialog and ProxyConfigurationActivity.

The correct location for this method should be Utils, and while we're on it we could directly close the resources correctly.

Attachments (0)

Change History (5)

comment:1 Changed on 11/06/2014 at 01:06:26 PM by rjeschke

  • Review URL(s) modified (diff)

comment:2 Changed on 11/24/2014 at 05:10:14 PM by rjeschke

  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed on 11/25/2014 at 09:03:33 AM by philll

What could possibly be tested here?

comment:4 Changed on 11/25/2014 at 11:00:17 AM by rjeschke

It could be tested if long texts in the app (like in the about dialogue) still appear correctly.

comment:5 Changed on 02/23/2015 at 01:12:08 PM by rjeschke

  • Milestone set to Adblock-Plus-for-Android-1.3

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from rjeschke.
 
Note: See TracTickets for help on using tickets.