Ticket #324 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Upload control bug in all IE versions

Reported by: dd Owned by: mpo
Priority: critical Milestone:
Component: < Upload Control Version: trunk
Keywords: Upload control Internet Explorer IE Cc: kauri-discuss@…

Description (last modified by dd) (diff)

This problem has been discovered in r1510, and is still present in trunk.

I used IE7 as browser. In IE8 the sample does appear to work.

Attachments

IE-upload.jpg (62.4 KB) - added by dd 3 years ago.
FF-upload.jpg (108.1 KB) - added by dd 3 years ago.
FF-selection.jpg (104.2 KB) - added by dd 3 years ago.

Change History

comment:1 follow-up: ↓ 2 Changed 3 years ago by freya

Hello,

can you specify in detail what problem you have with the upload control? The sample as well?
I have no problem in IE8, did not test other versions now.

comment:2 in reply to: ↑ 1 Changed 3 years ago by dd

  • Description modified (diff)

Replying to freya:

Hello,

can you specify in detail what problem you have with the upload control? The sample as well?
I have no problem in IE8, did not test other versions now.

comment:3 in reply to: ↑ description Changed 3 years ago by freya

Replying to dd:

This problem has been discovered in r1510, and is still present in trunk.

I used IE7 as browser. In IE8 the sample does appear to work.

comment:4 in reply to: ↑ description Changed 3 years ago by freya

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

Thanks for reporting. This should be fixed in r1539.

This problem has been discovered in r1510, and is still present in trunk.

I used IE7 as browser. In IE8 the sample does appear to work.

comment:5 Changed 3 years ago by dd

  • Status changed from closed to reopened
  • Resolution fixed deleted

The sample is fixed indeed. However, my usecase continues to fail. The difference between both is that i use a commit button. Replace the uploadform in the xml by

<form id="uploadForm" method="get" action="">

<fieldset>

<span kauri-idref="upload" kauri-type="file" kauri-control="upload-control"/>

</fieldset>
<input type="submit" value="Verzenden" />

</form>

to generate this error: 'this.elementIndex[...] is empty or not an object (267, 13)

comment:6 Changed 3 years ago by dd

This error apparently appears on some other pages as well, even when no upload control is used on the page.

comment:7 Changed 3 years ago by dd

I forgot to mention that the error on the other pages is something the last revision has broken, and this error did not occor in r1538.

comment:8 Changed 3 years ago by freya

If I use your snippet, everything loads and posts just fine, in FF, IE7 and IE8. (did not check IE6, is it IE6 you are using?)

Do you have this error when posting a form or while loading it? On what other pages you have this? And the samples?

Changed 3 years ago by dd

Changed 3 years ago by dd

Changed 3 years ago by dd

comment:9 Changed 3 years ago by dd

I have rechecked, and nothing has changed. The upload control does not work when
<input type="submit" value="Verzenden" />
is added to the sample. It fails both in IE7 as FF (see attachments). Also, to show r1539 has broken pages that use to work such as the sample of the collection control in FF, i added a screenshot of that as well. This page does still work in IE7 however.

comment:10 Changed 3 years ago by freya

OK thanks, should be bugfixed now. r1539 contains changes regarding the indexing mechanism of the forms, some changes will follow later today.

comment:11 Changed 3 years ago by freya

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.