Fixed Attachments and merging post issues

[gldtn]

Sorry if this has already been reported, but I ran into theses issues while dealing with attachments on 1.3..

If I merge a post that has attachments to another post that has reached the character limit, it gives me a character limit(1000) error and halts from merging the post, but the attachments still gets copied/carried over to the post I pointed the original post to merge to.



If you go to the post that the attachments got copied/carried to and delete the attachments from there, the attachments from the original post also get's deleted.



Also while attaching images and clicking to insert all as thumbnails, I often get the first attachment inserted only. Meanwhile if I remove the first inserted thumbnail from the rich text editor and click on insert all as thumbs again, it works as it should.



Hope that makes since

 

[Mike]

Am I correct in thinking your tables aren't InnoDB? This is all done in a transaction, so if it fails like this, it should be rolled back.

That said, I have made some tweaks here.

 

[gldtn]

Your right, for some reason most of my table were not innoDB, so I switched them following post here: http://xenforo.com/community/threads/changing-tables-from-myisam-to-innodb.31206/

Tried duplicating the bug above and it worked as it should

Thanks!