Hello, guys. When i edit any team, in Team listing is created a new item for same team. It's duplicated after every update. It's not a new team really, on every link it's the node number 16.

What can i do?

Thanks.

CommentFileSizeAuthor
#1 storm-team-list-dups-2.patch571 bytesjuliangb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juliangb’s picture

Status: Active » Needs review
FileSize
571 bytes

I've attached a patch that should fix this issue.

Status: Needs review » Needs work

The last submitted patch, storm-team-list-dups-2.patch, failed testing.

juliangb’s picture

Status: Needs work » Needs review

#1: storm-team-list-dups-2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, storm-team-list-dups-2.patch, failed testing.

dvasquez’s picture

-(+)1 :(

juliangb’s picture

Version: 6.x-2.1 » 6.x-2.x-dev

I don't know why this is failing.

Double checking against the latest version.

juliangb’s picture

Status: Needs work » Needs review

#1: storm-team-list-dups-2.patch queued for re-testing.

juliangb’s picture

Any chance of someone testing this patch?

I'd love to have the fix included in the download.

dbt102’s picture

I had several teams created which showed multiple duplicates as well. I had not been working with Storm for a while (focused on Project Management Module) so I made sure I did a - git pull origin - to work from most recent updates ( HEAD is now at 51091c4 Changelog update)

At this point I went in and edited the teams that had duplicate entries and saved. When saving the duplicate entries went away, and only one team was displayed. Repeated this for all duplicate teams and it worked OK. It appears the issue was resolved with the recent code updates.

For the record, I downloaded the patch from #1 above for testing.

Patch applied cleanly. Created new teams OK. Edited the new teams and this worked OK also, - no duplicates were created per original issue. Looks good.

(Sorry I didn't see this earlier, I had been keyed into the PM module - D7 - progress and not been paying attention too much to D6 Storm. Will watch more closely as we get close to Alpha release for PM and the Storm-Migration module to PM ! )

juliangb’s picture

That's interesting that the issue goes away without this patch.

Could you just test one more thing? (Since I'm not keen to change code if in fact there is nothing wrong).

Could you create a new revision of a team, and ensure it has multiple team members included? That should trigger the duplicates if anything does.