It looks like the latest version of Image breaks image attach weight. My attached images are being rendered as the very last item in both teaser and node view.

CommentFileSizeAuthor
#9 561362-image_attach-cck-weight.patch2.02 KBjoachim
#4 image_attach_weight.patch2.82 KBDavid Stosik

Comments

joachim’s picture

joachim commented
Status: Active » Postponed (maintainer needs more info)

You mean CCK field weight?

modctek’s picture

modctek commented

Yes. In both teaser and node view.

joachim’s picture

joachim commented
Status: Postponed (maintainer needs more info) » Active

I've realized that image attach already had weights available, in the content type admin -- we should perhaps have removed these when we added the CCK-type ones.

David Stosik’s picture

David Stosik commented
Version: 6.x-1.0-alpha6 » 6.x-1.x-dev
Assigned: Unassigned » David Stosik
Status: Active » Needs review
StatusFileSize
new image_attach_weight.patch2.82 KB
joachim’s picture

joachim commented
Status: Needs review » Fixed

Committed, thanks! :D

modctek’s picture

modctek commented

Was this fixed in the latest beta release (1.0-beta1)? I just installed and it does not appear to be fixed. I was also unable to apply the above patch to alpha6.

joachim’s picture

joachim commented

Nope, the beta was release a few days ago and the fix is dated today.
The patch is against CVS HEAD.
You can try the dev release if you like.

modctek’s picture

modctek commented

Just downloaded the dev release and updated my website, but image weight is still being disregarded. Perhaps the changes in CVS head haven't made it to the downloadable dev version? Installation occurred about 5 mins prior to this posting.

joachim’s picture

joachim commented
StatusFileSize
new 561362-image_attach-cck-weight.patch2.02 KB

Indeed -- we fixed one thing but didn't implement the CCK api stuff properly.
Committing this patch.

And a thank you to signup module for the example!

modctek’s picture

modctek commented

This is still not fixed for me on one of my sites, but others using the same code-base are properly weighting images. Perhaps another module is conflicting?

joachim’s picture

joachim commented

This fix was applied after the beta 2 release. It is therefore only in CVS and the latest dev release.
In the Changelog file you will find a list of all fixes with their issue number.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

asb’s picture

asb commented
Version: 6.x-1.x-dev » 6.x-1.0-beta3
Status: Closed (fixed) » Fixed

> This fix was applied after the beta 2 release. It is therefore only in CVS and the latest dev release.

...and it is included in the 6.x-1.0-beta3 release (setting status to "fixed").

Greetings, -asb

limex67’s picture

limex67 commented

6.x-1.0-beta3 is from Sep-13. Your "fixed" posting was on Sept 27 which is after the release date of beta3

Are you sure that beta3 got the fix included?
Because I still have the issue with beta3.
Regards

asb’s picture

asb commented

Hi,

I had the issue with image 6.x-1.0-beta2, after upgrading to 6.x-1.0-beta3 it is gone on all my D6 installations using "Image" module. That's all I can tell you. If you want to know it exacter you'll have to look into the code.

Have you checked how your fields are arranged at ./admin/content/node-type/article/fields (or whatever content type you're trying to modify)? Where is the "Attached images" field located in relation to the "Node body" field?

Greetings, -asb

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.