This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 130427 - I18N - need fix for 130235 for refactoring
Summary: I18N - need fix for 130235 for refactoring
Status: RESOLVED INVALID
Alias: None
Product: editor
Classification: Unclassified
Component: Refactoring (show other bugs)
Version: 6.x
Hardware: Sun Windows XP
: P3 blocker (vote)
Assignee: Daniel Prusa
URL:
Keywords: I18N
Depends on:
Blocks:
 
Reported: 2008-03-18 16:46 UTC by Ken Frank
Modified: 2008-04-15 12:40 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
image (133.86 KB, image/gif)
2008-03-18 18:39 UTC, Ken Frank
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ken Frank 2008-03-18 16:46:45 UTC
see 130235 - Maros requested I file issue for refactoring for that situation.
Comment 1 Jan Becicka 2008-03-18 17:01:15 UTC
For what situation? Issue 130235 is fixed.
Please follow rules. Thanks. http://qa.netbeans.org/bugzilla/rules.html
Comment 2 Maros Sandor 2008-03-18 17:22:50 UTC
AFAIK Ken tested the scenario in Refactoring and characters did not display correctly, you should probably attach those
screenshots here.
Comment 3 Ken Frank 2008-03-18 18:38:58 UTC
to clarify 
there might be 2 separate things here

1. to do the fix for 130235 for refactoring; Maros can give you the info on that.

2. but also, on mac, was seen what is in attached gif, which may or may not be
related to the fix for 130235 - if its separate, a separate issue can be filed.

ken.frank@sun.com
Comment 4 Ken Frank 2008-03-18 18:39:33 UTC
Created attachment 58608 [details]
image
Comment 5 Ken Frank 2008-03-18 18:44:57 UTC
more on the case for mac

local history and the refactor
preview diff view does not show the multibyte ok for one of the 2 diff windows
of each.


using utf-8 or mac-roman(an euc-jp) encoding -

local history - current view mbyte is ok; previous one is not

refactor preview - current view is ok; to be refactored window is not

attached is gif.
Comment 6 Daniel Prusa 2008-03-20 09:57:11 UTC
I will try to adopt the fix from local history.
Comment 7 Jan Becicka 2008-03-26 09:29:29 UTC
This is not P2 according bug priority guidelines. Just incorrect visual appearance. Anyway Dan will fix it for 6.1.
Comment 8 Daniel Prusa 2008-03-27 18:03:30 UTC
I have investigated the related code. I have found there is no need to make a fix for issue 130235 in refactoring
preview. Local history and the versioning modules create temporary files when displaying diffs. There was a problem that
these files have not been associated with the proper encoding. On the other hand, refactoring preview does not store
content of new sources in temporary files, it writes the content into editor documents, so we do not need to handle
encoding like the local history does.

As for the second problem reported against Mac, it seems it is a different issue, perhaps something wrong in editor.
Issue 130235 reports missing Japanese characters in both views (old source view, new source view). When examining the
attached gif, there are only some Japanese characters (not all) missing in one view.
Have you managed to reproduce this problem also on another platform?
Comment 9 Ken Frank 2008-03-27 18:52:32 UTC
am marking as incomplete now; since we are talking just about the mac part,
I can't replicate this on new leopard installed os, using both utf8
or euc-jp project encodings. (was using 10.4 mac before)

I will keep looking at this , but for now, lets waive it or move to p3
unless incomplete keyword is enough to keep from dashboard for 6.1

ken.frank@sun.com
Comment 10 Jiri Prox 2008-04-11 01:48:41 UTC
moving opened issues from TM <= 6.1 to TM=Dev
Comment 11 Ken Frank 2008-04-14 22:06:58 UTC
based on below comments, can this one be closed, if the
fix related to 130235 is not needed here ?

ken.frank@sun.com
Comment 12 Jan Pokorsky 2008-04-15 12:40:01 UTC
Sure, closing as INVALID since it was not a refactoring bug.