Comments

Heine’s picture

A few bugs / compatability issues have been resolved:

#295657: JS bug after preview comment
#295645: Error after delete node

I've not yet received any bugs regarding the upgrade process from earlier versions of comment upload.

I did forget to include one other 'thing' that needs to happen before a 6.x-1.0 release. Doh!

The new list:

  • Views 2 integration (already working on)
  • Test the upgrade path
  • Test the module dev release
Gábor Hojtsy’s picture

Looks like people reported several upgrade problems so far. #326668: No *old* comment uploads shown after 5.12 -> 6.6 upgrade which is about files disappearing altogether in the upgrade my understanding. #297845: Files attached to comments after upgrade are missing which is about files being listed but non-existent after upgrade. These might be the same bug differently worded (and I misunderstood the difference).

vladimir.dolgopolov’s picture

I've provided a patch for #326668: No *old* comment uploads shown after 5.12 -> 6.6 upgrade.
The issue is due to using drupal_write_record() instead of update_sql() in comment_upload.install.
Not sure it is the same as #297845: Files attached to comments after upgrade are missing so {comment_upload} is not empty.

Gábor Hojtsy’s picture

Issue tags: +drupal.org upgrade
Rosamunda’s picture

subscribing

Gábor Hojtsy’s picture

Issue tags: -drupal.org upgrade

It was tested a lot in Cambridge by Derek and Austin for the drupal.org upgrade. Worked with JS and without JS. Maybe need more testing, but removing the upgrade tag, since we found it fine.

rcross’s picture

anything still stopping this from having a 1.0 release?

dww’s picture

Just updated the issue summary with my assessment of the issues that are still blocking a 6.x-1.0 release.

dww’s picture

Issue summary: View changes

updated list of remaining issues

dww’s picture

Issue summary: View changes

added #1461242