Announcement

**mpetransky** · 3 Feb 2012, 03:56

Guys, any news on this, need to have that fixed asap.

**Isomorphic** · 3 Feb 2012, 04:54

We're investigating, please bear with us

**Isomorphic** · 3 Feb 2012, 06:58

Can you show the code for a minimal form that causes this problem? We are trying to to reproduce the problem, but so far have not been able to.

**Isomorphic** · 3 Feb 2012, 07:32

Also, which browser(s) are you seeing this problem with? We are beginning to suspect that the problem is browser-specific.

**mpetransky** · 3 Feb 2012, 11:19

Hi,

this is the form, pretty simple. SMTextItem and SMTextArea directly extend TextItem and TextAreaItem and just use the composite constructor to set properties, no custom code there:

Code:

private DynamicForm initForm() {

		DynamicForm form = new DynamicForm();

		// Text Fields
		SMTextItem number = new SMTextItem("number", Utils.getTexts().numberAcronym(), 8, true);
        SMTextItem locationObject = new SMTextItem("locationObject", Utils.getTexts().locationObject(), 255, true);
		SMTextItem functionalUnit = new SMTextItem("functionalUnit", Utils.getTexts().functionalUnit(), 255, true);
		SMTextItem responsible = new SMTextItem("responsible", Utils.getTexts().responsible(), 255, true);

		// Text Areas
		SMTextAreaItem defect = new SMTextAreaItem("defect", Utils.getTexts().defect(), 300, 150, false, true, 1);
		defect.setTitleStyle("rc-formItemTitle");
		defect.setRequired(true);
		defect.setTitleVAlign(VerticalAlignment.TOP);

		// Image Upload Item
		FileItem imageUploadItem = new FileItem("imageData", "<b>" + Utils.getTexts().image() + "</b>");
		imageUploadItem.setTitleStyle("rc-formItemTitle");

		form.setFields(number, locationObject, functionalUnit, imageUploadItem,	defect, responsible);

		return form;

}

Browsers i've checked are: Chrome 16, FF 6, IE 6, IE 9.

**Isomorphic** · 6 Feb 2012, 14:52

We eventually got to the bottom of this, and it turns out to be a known and documented issue, although not one that is really confined to the use of FileItem. However, the article documenting this is not present in the SmartGWT docs because there are elements of it that are not applicable to SmartGWT. We will rework things to make sure that relevant important information is present; in the meantime, please follow the advice on Content Encodings in this equivalent SmartClient article (but ignore the rest of the article): http://www.smartclient.com/docs/8.2/...ml#group..i18n

**mpetransky** · 7 Feb 2012, 01:46

Hi Isomorphic,

i have the contentType page directives as well as meta headers on every page, i have also added this now to the DSResponse that i'm sending back from the dataSource, but it didn't help.

Code:

response.setExportHeader("Content-Type: text/html; charset=UTF-8");

P.S.:

Just to make sure we are on the same line here, it's the text fields that are broken, not the file itself.

**Isomorphic** · 7 Feb 2012, 02:19

So, if you look at the response for your bootstrap .html or .jsp file using something like Fiddler, does it have a content-type header specifying charset=UTF-8?

**mpetransky** · 7 Feb 2012, 04:26

Yes, this is from firebug:

Code:

Content-Type	text/plain; charset=utf-8

this is the raw response:

Code:

//isc_RPCResponseStart-->[{queueStatus:0,isDSResponse:true,invalidateCache:false,status:0,data:[{imageData:null,pointId:4,defect:"Mangel DE",finished:false,functionalUnit:"q",id:6,locationObject:"q",number:"q",priority:null,responsible:"q",solution:"Losung DE"},{imageData:null,pointId:4,defect:"Mangel DE",finished:false,functionalUnit:"asd",id:8,locationObject:"asd",number:"asd",priority:null,responsible:"asd",solution:"Losung DE"},{imageData:null,pointId:4,defect:"bootstrap ",finished:false,functionalUnit:"durchgef�ƒ�’�†�€™�ƒ�€ â�‚��„��ƒ�’â�‚�� �ƒ¢â�€š¬â�€ž¢�ƒ�’�†�€™�ƒ¢â�€š¬�…¡�ƒ�’â�‚�š�ƒ�€š�‚¼hrten Teilaudits k�ƒ�’�†�€™�ƒ�€ â�‚��„��ƒ�’â�‚�� �ƒ¢â�€š¬â�€ž¢�ƒ�’�†�€™�ƒ¢â�€š¬�…¡�ƒ�’â�‚�š�ƒ�€š�‚¶nnen ",id:9,locationObject:"sdf",number:"sdf",priority:null,responsible:"sdf",solution:"Deutschland grenzt an neun europ�ƒ�’�‚¤ische Nachbarstaaten; naturr�ƒ�’�‚¤umlich grenzt es an die Gew�ƒ�’�‚¤sser der Nord- und Ostsee und im S�ƒ�’�‚¼den an das Bergland der Alpen. Es liegt in der gem�ƒ�’�‚¤�ƒ�’�…¸igten Klimazone und z�ƒ�’�‚¤hlt mit rund 81,8 Millionen Einwohnern zu den dichtestbesiedelten Fl�ƒ�’�‚¤chenl�ƒ�’�‚¤ndern der Erde."},{imageData:null,pointId:4,defect:"Unseres Erachtens werden nicht\ngenügend Kontrollen durchgeführt oder die nötigen ",finished:false,functionalUnit:"sdf",id:10,locationObject:"sdf",number:"sdf",priority:"P1",responsible:"sdf",solution:"Deutschland grenzt an neun europ�ƒ�’�‚¤ische Nachbarstaaten; naturr�ƒ�’�‚¤umlich grenzt es an die Gew�ƒ�’�‚¤sser der Nord- und Ostsee und im S�ƒ�’�‚¼den an das Bergland der Alpen. Es liegt in der gem�ƒ�’�‚¤�ƒ�’�…¸igten Klimazone und z�ƒ�’�‚¤hlt mit rund 81,8 Millionen Einwohnern zu den dichtestbesiedelten Fl�ƒ�’�‚¤chenl�ƒ�’�‚¤ndern der Erde."},{imageData:null,pointId:4,defect:"Die haupts�ƒ¤chlichsten M�ƒ¤ngel sind nach wie vor im organisatorischen Brandschutz in den internen Bereichen anzutreffen. Es sind deshalb unbedingt verst�ƒ¤rkte organisatorische Massnahmen zu ergreifen, vor allem in Bezug auf Brandverh�ƒ¼tung und allgemeine Ordnung.\n\nWegen Platzknappheit sind in mehreren Technikr�ƒ¤umen, die nach den Vorschriften der Feuer-polizei allgemein frei von Waren frei zu halten w�ƒ¤ren, weiterhin gr�ƒ¶ssere Brandlasten vorhanden.",finished:false,functionalUnit:"q",id:11,locationObject:"q",number:"q",priority:null,responsible:"q",solution:"Die haupts�ƒ¤chlichsten M�ƒ¤ngel sind nach wie vor im organisatorischen Brandschutz in den internen Bereichen anzutreffen. Es sind deshalb unbedingt verst�ƒ¤rkte organisatorische Massnahmen zu ergreifen, vor allem in Bezug auf Brandverh�ƒ¼tung und allgemeine Ordnung.\n\nWegen Platzknappheit sind in mehreren Technikr�ƒ¤umen, die nach den Vorschriften der Feuer-polizei allgemein frei von Waren frei zu halten w�ƒ¤ren, weiterhin gr�ƒ¶ssere Brandlasten vorhanden."},{imageData:null,pointId:4,defect:"Wegen Platzknappheit sind in mehreren Technikräumen, die nach den Vorschriften der Feuer-polizei allgemein frei von Waren frei zu halten wären, weiterhin grössere Brandlasten vorhanden.",finished:false,functionalUnit:"asd",id:12,locationObject:"asd",number:"asd",priority:null,responsible:"asd",solution:"a"},{imageData:null,pointId:4,defect:"and ä appears",finished:true,functionalUnit:"a",id:13,locationObject:"a",number:"a",priority:"P3",responsible:"asd",solution:"and ä appears"},{imageData:null,pointId:4,defect:"durchgeführten Teilaudits können 080-.DN� ",finished:false,functionalUnit:"durchgeführten Teilaudits können ",id:14,locationObject:"asd",number:"asd",priority:null,responsible:"asd",solution:"asd"},{imageData:null,pointId:4,defect:"durchgeführten Teilaudits können ",finished:false,functionalUnit:"asd",id:18,locationObject:"asd",number:"asd",priority:null,responsible:"asd",solution:"durchgeführten Teilaudits können "}],exportHeader:"Content-Type: text/html; charset=UTF-8"}]//isc_RPCResponseEnd

**Isomorphic** · 7 Feb 2012, 06:09

OK, thanks - we see the issue. There is a long-standing server-side bug in the framework that only shows itself under certain circumstances (which never crop up in our test and development environments, hence the trouble we've had pinning this down).

We have committed a fix for this, which will be present in tomorrow's 3.1d and 3.0p builds. Note that all that other stuff about setting charset and content-type headers is still required.

**mpetransky** · 7 Feb 2012, 07:35

Hi Isomorphic,

thank you! Btw. we are still using SmartGWT 2.5, will it be available there as well ?

**Isomorphic** · 7 Feb 2012, 08:13

Yes it will

**mpetransky** · 7 Feb 2012, 08:36

Ok, thank you.

**mpetransky** · 9 Feb 2012, 09:11

It works, thanks a lot!

Announcement

Broken response charset when using FileItem

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment