Merge lp:~vbkaisetsu/tomdroid/fix-1049026 into lp:tomdroid/beta

Proposed by Koichi Akabe
Status: Merged
Merged at revision: 467
Proposed branch: lp:~vbkaisetsu/tomdroid/fix-1049026
Merge into: lp:tomdroid/beta
Diff against target: 15 lines (+4/-1)
1 file modified
src/org/tomdroid/util/NoteXMLContentBuilder.java (+4/-1)
To merge this branch: bzr merge lp:~vbkaisetsu/tomdroid/fix-1049026
Reviewer Review Type Date Requested Status
Tomdroid Maintainers Pending
Review via email: mp+123705@code.launchpad.net

Description of the change

This branch fix lp:1049026
Please consider merging this branch.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/org/tomdroid/util/NoteXMLContentBuilder.java'
2--- src/org/tomdroid/util/NoteXMLContentBuilder.java 2012-09-07 07:41:54 +0000
3+++ src/org/tomdroid/util/NoteXMLContentBuilder.java 2012-09-11 10:05:19 +0000
4@@ -96,7 +96,10 @@
5 TreeMap<String,String> replacements = new TreeMap<String,String>();
6 replacements.put( "&", "&amp;" ); replacements.put( "<", "&lt;" ); replacements.put( ">", "&gt;" );
7 for( Map.Entry<String,String> entry: replacements.entrySet() ) {
8- for( int currPos=plainText.lastIndexOf(entry.getKey(), plainText.length()); currPos>0; currPos=plainText.lastIndexOf(entry.getKey(), currPos-1) ){
9+ for( int currPos=plainText.length(); currPos>=0; currPos-- ){
10+ currPos = plainText.lastIndexOf(entry.getKey(), currPos);
11+ if(currPos < 0)
12+ break;
13 noteContent.replace( currPos, currPos+entry.getKey().length(), entry.getValue() );
14 TLog.d(TAG, "new xml content: {0}", noteContent.toString());
15 }

Subscribers

People subscribed via source and target branches