Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
1. cck 6.x-2.1
2. imagefield 6.x-3.0-beta1
3. tmp read/write
4. public downloads
5. imagefield 6.x-3.0-alpha4 configured by ubercart, then updated to 6.x-3.0-beta1
6. imagefield image upload
7. IE6: image upload works only with javascript enabled (via ajax). image deletion works only with javascript disabled (via full request)
8. check this on IE6 on common config.
Comment | File | Size | Author |
---|---|---|---|
#28 | dim_length.patch | 4.53 KB | Island Usurper |
Comments
Comment #1
quicksketchThanks for the report. The image upload only working when JavaScript is enabled has been fixed in #403216: Upload fails when JS is disabled. I'll look into deleting in IE6. I'm guessing that you're implying that all other browsers work fine when deleting images?
Comment #2
quicksketchUsing the latest CVS version (with the #403216: Upload fails when JS is disabled patch applied), I'm not able to confirm this problem. IE6 works uploading/removing/saving images with or without JavaScript.
Comment #3
aks CreditAttribution: aks commentedFirefox works fine! I'll try latest CSV version to fix #403216: Upload fails when JS is disabled
Comment #4
quicksketchNote you'll need the latest version of FileField, which is where the no-JS bug actually originated.
Comment #5
aks CreditAttribution: aks commentedI've applied the patch to filefield, it had effect that IE is able to upload/delete without JS, but still can't delete with JS. It seems that ajax call works (imagefield item disappeared in list after clicking 'delete'), and real deletion does not.
Comment #6
quicksketchWhat do you mean by "real deletion"? The file isn't actually removed from the node or disk until the node is saved.
Comment #7
rsm08 CreditAttribution: rsm08 commentedI seem to have a similar problem, so I'll join this thread.
When adding or editing a node with a CCK imagefield (with multiple images allowed), :
In IE7, upload works fine, but when clicking the "Add another item" button to add fields for uploading another image, the whole imagefield part of the form just disappears.
The page doesn't reload, and there's no problem in Firefox, so it must be a javascript-issue. (Which leaves me clueless since I have no js debugger for IE).
I'm using:
Drupal 6.9
CCK 6.x-2.1
ImageField 6.x-3.0-alpha4
Hope you can help, thanks!
Comment #8
quicksketchrsm08, you'll need to upgrade to at least beta3 before we'll be able to help you. The problem you're reporting very well may have already been fixed. Upgrading to CCK 2.2 is also highly recommended, since it corrects an issue with the "more" button breaking uploads.
Comment #9
rsm08 CreditAttribution: rsm08 commentedquickstetch, thanks, I was trying to avoid the beta versions, but I'll try your advice and report my findings here.
Comment #10
drewish CreditAttribution: drewish commentedi've got to chuckle at being willing to use alpha software but being scared off by beta
Comment #11
rsm08 CreditAttribution: rsm08 commentedOh, I didn't notice it was an alpha version. ;) What I meant was that I was trying to avoid dev versions if there was a safe but earlier version available!
Well, I updated, and though the upload interface is now nicer and simpler than before, the javascript problem remains the same. The instance I click "Add another item", the whole imagefield area disappears without a page reload.
Note that the problem is still only in IE7. FF works fine.
My versions are now:
CCK 6.x-2.2
ImageField 6.x-3.0-beta3
ImageAPI 6.x-1.5
ImageCache 6.x-2.0-beta8
Ubercart 6.x-2.0-beta5
Hope you can help!
Comment #12
quicksketchAre you having trouble with Ubercart ImageFields, or is it all ImageField's in general? I know Ubercart had to do some changes when the beta versions of ImageField came out, see #404352: Image (field_image_cache) is an inactive image field.
I ask because I can't reproduce this problem at all. On my sandbox uploading/removing/add another item all work just fine on IE7 and even on IE6. Are you logged in as the same user in both FF and IE?
Comment #13
rsm08 CreditAttribution: rsm08 commentedActually it's only Ubercart imagefields. I tried this in another content type, and there was no problem.
So it's only in Ubercart's content types that this problem occurs. I guess I'll have to take this issue to Ubercart's site then.
Thanks for helping me out!
Comment #14
quicksketchGreat, thanks for the followup. It seems like the Ubercart folks have fixed this just recently in that issue I posted above.
Comment #15
rsm08 CreditAttribution: rsm08 commentedBut this issue you posted, I can't see how this has anything to do with my problem!?
As I understand it, the mentioned issue has something to do with problems caused by updating imagefield, and I had this problem before I updated either module!
Am I missing something here?!
Comment #16
chumm CreditAttribution: chumm commentedI am experiencing the same issue with IE7 and IE8.
In all browsers (IE7, IE8, Firefox), clicking the 'REMOVE' button associated with the image does appear to remove the image from the edit page. Then as usual, when clicking 'SAVE' at the bottom, the product page shows up once again. However, if using IE, the image is not deleted and still displays.
This is NOT the case with Firefox. Firefox works as it should.
Can anyone offer some help with this, as I need it to work in IE. I tried searching the forms for can't delete images ie and didn't see anything relevant to my issue.
For reference, I'm running the following which Drupal reports are all up to date:
Drupal 6, Ubercart 2
Content 6.x-2.2
FileField 6.x-3.0-rc1
ImageField 6.x-3.0-rc1
ImageAPI 6.x-1.5
ImageCache 6.x-2.0-beta8
Comment #17
chumm CreditAttribution: chumm commentedI meant to re-open this in my last post.
Comment #18
quicksketchI'm at a loss for what's wrong here. Any other Ubercart users please post if you're having trouble or what can be done to solve the problem. I'm leaving this up to the Ubercart team to solve this problem, since I'm not going to install an entire e-commerce site just to test out FileField. It works fine on normal node posts in all browsers.
Comment #19
chumm CreditAttribution: chumm commentedIs there anything that I could do to try and debug? I'm surely not a programmer but would be willing to help out to test/debug this if it can help lead to a fix.
Comment #20
chumm CreditAttribution: chumm commentedI tried a completely fresh Drupal and Ubercart install. This problem continues to exist! My experiences below:
Installed Drupal 6.9 through my webhost. Then I immediately updated it to v6.10.
Then I uploaded all the Ubercart files (v6.x-2.0-beta5) and installed the latest Ubercart modules. I went to enable image support and got the following error messages:
Then I uploaded all the latest Ubercart dev version files (6.x-2.x-dev) and ran the update.php script.
Then I went to enable image support once again and this time got the following errors:
Interestingly enough, it does say: Product image support has been automatically configured by Ubercart.
HOWEVER.... I am experiencing the exact same image problems as I am on my primary site. Images can be removed from products only when using Firefox. Doesn't work in IE. Also, my product images are missing the 'replace' image upload option.
Does anybody know what is going on here? This is quite frustrating.
Comment #21
quicksketchComment #22
chumm CreditAttribution: chumm commentedUpdate: Reverting to FileField 6.x-3.0-alpha7 and ImageField 6.x-3.0-alpha4 fixes the issue, and I can once again actually delete images that are associated with products.
Perhaps the developers of these modules can figure out what happened somewhere between these versions and the current ones.
Comment #23
Island Usurper CreditAttribution: Island Usurper commentedOh blargh.
I think it's got something to do with the product node form. There's nothing different (any more) about the image fields that Ubercart puts on products. I can add the existing field to other content types, and it works fine in IE (I'm using IE8). But, adding a new image to pages, and then adding that field to a product class causes the Remove button and "Add another item" to not work.
If someone can test with another node-type module (Event, maybe?), I'd like to know if that's the problem. I don't know what is unique about uc_product_form() that throws AHAH in IE for a loop.
Comment #24
Island Usurper CreditAttribution: Island Usurper commentedComment #25
CandC540 CreditAttribution: CandC540 commentedI have the same problem with attaching multiple files breaking FileField
Comment #26
epop CreditAttribution: epop commentedjoining this thread, i too cant delete images from products in IE. I have all the latest versions. This is a major issue. Other content types work fine its just Ubercart Products. So the problem must be with Ubercart and its interactions wit IE.
Comment #27
Island Usurper CreditAttribution: Island Usurper commentedThis is an amazing bug. I'm really fortunate that the IE 8 JScript debugger works even better than Firebug, or I might never have found it.
Some background: ahah.js uses a function called ajaxSubmit() from jquery.form.js to send the HTTP request. ajaxSubmit() takes the data it is given, and combines it with the form values on the page to send as POST data. And in the vast majority of cases, this works just fine.
However, to get the form values, ajaxSubmit() calls formToArray(). This function runs a for() loop on all of the elements of the form, and uses the els.length as the upper bound of the loop. Somehow, and only in IE, els.length isn't the numeric size of the array of elements, but the actual "length" form element. You can't compare objects as numbers, so the function returns an empty array, and the form values aren't included in the POST data.
When Filefield can't find $_POST['form_build_id'], it thinks the upload failed because the file was too big, and returns an error message saying so.
So, the most obvious solution is to change the name attribute of the length field. Because I like symmetry, I also changed the width and height fields. This should only affect the node-edit form. $node->length and friends should remain the same since uc_product_load() wasn't changed.
Comment #28
Island Usurper CreditAttribution: Island Usurper commentedAnd with a patch, this time.
Comment #29
fenstratWorks perfectly.
Very, very strange bug.
Lyle your spot on - the culprit is the for loop which uses els.length as the upper bounds. The reason looks to be IE's reserved words. In particular length.
Interesting timing, I'd finally got some time to track this down and was just submitting a patch to change length to dimension_length just as you did (minus your consistency touch with width and height). Last minute search found this - you beat me too it, so still can't clame my first patch!
btw this also effects Opera.
Making these 2 as duplicates:
#448314: Unable to remove product images in IE or Opera
#435312: Getting error message when deleting images
Comment #30
Island Usurper CreditAttribution: Island Usurper commentedThanks for testing. Committed.
Comment #32
asianguy CreditAttribution: asianguy commentedwhen UPS product description Package type is set to Customer Supplied Package, Product cant be saved because "Dimensions are required for custom packaging", it always ask for dimension even if dimension has values
Comment #33
5n00py CreditAttribution: 5n00py commented#757724: Dimensions reset Look to this bug pls.
Patch from #28 NOT "Works perfectly."
Comment #34
5n00py CreditAttribution: 5n00py commentedPatch from #404234-28: Ajax bug on upload/delete Ubercart images create new bug.
Patch store dimensions data in:
$node->dim_length,
$node->dim_width,
$node->dim_height
instead
$node->length
$node->width
$node->height
This looks good, but.
Function uc_product_load return data from db directly to node object and still use old variable names.
I see 3 possible ways to fix this:
I think storing data from 'length' db column in variable named dim_length not good idea.
--
Sorry for my English :)
Comment #35
Summit CreditAttribution: Summit commentedSubscribing, greetings, Martijn
Comment #36
longwaveThe original issue is fixed. The dimensions bug is open as another issue.