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.
Summary: | org.openide.text.DataEditorSupport$1: The file C:/projetos/mercedes-migracao/mercedes-migracao-service/src/main/java/br/com/tecsinapse/dealerprime/migracao/service/CarteiraService.java cannot be safel | ||
---|---|---|---|
Product: | platform | Reporter: | misterm <misterm> |
Component: | Text | Assignee: | Miloslav Metelka <mmetelka> |
Status: | NEW --- | ||
Severity: | normal | CC: | abien, AFuchs, alexferreiragpe, amobilia, atlioddsson, deep_pl, elburgl69, emiddio, FrantaM, gameshas, ggghhhjjj, gtzabari, IrianR, jag, javatlacati, jomello_br, jyeary, khansudarsan, kRk, legitalk, Lolmewn, MackSix, marckoby, misterm, mlsprtc, mosca_morta, msd575, tboudreau, terje7601, tskillman, vv159170, zdenek.rod |
Priority: | P2 | Keywords: | 8.1_WAIVER_APPROVED |
Version: | 7.4 | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | 204365 |
Bug Depends on: | 255122 | ||
Bug Blocks: | |||
Attachments: |
stacktrace
stacktrace stacktrace stacktrace stacktrace stacktrace stacktrace stacktrace stacktrace stacktrace stacktrace |
Description
misterm
2013-10-04 12:49:22 UTC
Created attachment 140803 [details]
stacktrace
This bug already has 5 duplicates see http://statistics.netbeans.org/exceptions/detail.do?id=204365 Created attachment 144047 [details]
stacktrace
changes character set in project
Created attachment 148749 [details]
stacktrace
From the exception message, it looks like this exceptiion was supposed to result in a dialog being shown asking me if the file should be opened in UTF-8.
This bug already has 20 duplicates see http://statistics.netbeans.org/exceptions/detail.do?id=204365 Created attachment 149829 [details]
stacktrace
Opening a file with bad encoding - the exception should have been a question to the user.
Created attachment 153804 [details]
stacktrace
Clearly supposed to show a dialog, judging from the exception message. Maybe used to be UserQuestionException or something?
Created attachment 154325 [details]
stacktrace
Trying to open a class with encoding problems from a maven build failure link.
Created attachment 154573 [details]
stacktrace
Clicked on link warning in Run output window of Maven JSF project.
com/jsfcompref/Model.java:[50,38] unmappable character for encoding UTF-8
Created attachment 155104 [details]
stacktrace
Clicked on link in output window that said "com/jsfcompref/trainer/entity/accessor/UserRegistry.java:[65,42] unmappable character for encoding UTF-8"
Created attachment 155590 [details]
stacktrace
Attempting to do a maven build.
The current behavior was intended, and is ensured by several tests. The fix would be with risk and, as issue 255122 plans to rewrite the document handling, short-lived I would like to ask for a waiver for 8.1. The bug here is that that exception's message should be appearing in an OK/Cancel dialog. There should be some code that catches the exception and shows the dialog (the original behavior), and probably that code was deleted at some point. (In reply to _ tboudreau from comment #13) > The bug here is that that exception's message should be appearing in an > OK/Cancel dialog. There should be some code that catches the exception and > shows the dialog (the original behavior), and probably that code was deleted > at some point. The tests explicitly test for this not to happen when a document is loaded. Only when the UQE is thrown during reload, the dialog is shown. Unfortunately I don't remember if we used to be showing the dialog or not in the past :( But if we show it during the reload it makes sense to show it during the load as well. I agree with Ralph that making a change that would popup a dialog during file loading process at this point could be dangerous and we should probably resolve it once the 8.1 is frozen. Although there are many tests any change in openide.text that involves threading should ideally be done soon in dev cycle to reveal potential problems. OK. Let's fix this in trunk after 8.1 branch is created and if the fix will not cause any regressions we could deliver it in the first 8.1 patch release. Bug waiver for 8.1 approved. Created attachment 157770 [details]
stacktrace
Happened during build
Bug still present on dev 201701020001 Created attachment 163835 [details]
stacktrace
.
|