zikula-modules / DizkusModule

Official repository for Dizkus, a fully integrated forum solution for Zikula 1.4+
21 stars 7 forks source link

Topic hook comment count is one off #261

Closed cmfcmf closed 10 years ago

cmfcmf commented 10 years ago

bildschirmfoto vom 2014-08-30 20 14 51

craigh commented 10 years ago

that might be just from data corruption.

cmfcmf commented 10 years ago

No, I rather think it's because the first post is not shown (the one saying cmfcmf add the extension blabla).

craigh commented 10 years ago

see here: http://zikula.org/index.php/forum/topic/59771

craigh commented 10 years ago

as I said, I think your issue was from data corruption. Here is a new entry.

screen shot 2014-09-01 at 9 06 53 am

cmfcmf commented 10 years ago

Please try to add another version to this extension and see if the comment count still says 1.

craigh commented 10 years ago

you are correct, of course. but I think this is a product of how EL and Dizkus are working together (or not). I think we need to take a look at options here.

EL is obviously maintaining one hooked topic for the Extension and adding 'release announcements' to the thread. This is broken too - it is only copying the _first_ announcement instead of adding the new version number, etc. to the thread. this must be corrected. Finally, the count doesn't include any release announcements so that must be looked at also.

Another option would be to figure out a way to force a NEW topic for each release. I'm not sure if this is possible.

either way - this must be improved, but I think it is EL's problem, not Dizkus.

screen shot 2014-09-01 at 9 16 56 am

craigh commented 10 years ago

fixed