Copy card: "show as thumb" attachment setting is not cloned about wekan HOT 2 OPEN

I may be wrong but I think the trouble code is there:

When a card is inserted, there may be a control that disable this option to show 'attachements' if the card has no attachment. then attachment are created in the next block. But then should try a Cards.update(this) or something more sweet/targeted that retry to set this setting.

from wekan.

@xet7

OK, I got it.

When the card is inserted, the coverId key of the new card points to the previous attachment file _id.

When attachments have been copied, one should look for the _id of the new card new attachments whose key "meta.copyFrom" is equal to the current coverId key. Then, change this key, with this _id.

The trouble is copyFile is async. The best would be copyFile to return the _id of the new attachment in a Promise, and to wait for this Promise only for the copy of the cover attachment. I didn't go further because it would imply waiting, and I know wekan is really concerned with scalability (copying a board means possibly copying a lot of cards).

Another way would be copyFile to handle this coverId change when called from the copy card function... Sounds strange...

We could also consider, the situation completely differently and handle it at display time. If coverId points to an attachment not belonging to the current card, look for current card attachments whose "meta.copyFrom" is equal to the current coverId and then correct it. This should be efficient. And if not enough, we could add a flag on the new card after copy to trigger this only once, but I think this is needless. When displaying a card anyway, the Card object is built with all the data we need, and it's no more than a condition test.

Your opinions?

from wekan.