Opened 9 years ago

Closed 7 years ago

Last modified 6 years ago

#1297 closed bug (fixed)

comment funtcion broken

Reported by: kiwiflave Owned by:
Priority: critical Milestone: Legacy Resolved
Component: BW Profile Keywords: comment function
Cc:

Description

There seems to be a bug with the comment function (that s what I get: http://www.bewelcome.org/members/steinwinde/comments/add).

BW Rox version: user agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.0.14) Gecko/2009082707 Firefox/3.0.14 (.NET CLR 3.5.30729) request uri: http://www.bewelcome.org/members/steinwinde/comments/add

Change History (7)

comment:1 Changed 9 years ago by fake51

Should be fixed with r9221

comment:2 Changed 9 years ago by jeanyves

I think there is another problem: the $IdMember? variable is useless

I propose a fix with also two addition logs in the model (in case people report problem about it, because it looks to me very strange that this big was not reported before)

comment:3 follow-up: Changed 9 years ago by jeanyves

  • Summary changed from comment funcion broken to comment funtcion broken
  • version changed from production to alpha

My two changes are to test on addcommentpage and addcommend.column3

@Peter, I still see the bug in production. I will move my additional fix in Alpha where it is to be tested

Question why do we only see this bug now ?

comment:4 Changed 9 years ago by fake51

You're seeing a different bug that the one I fixed. And don't move anything yet, your code was bad, I had to update it

comment:5 in reply to: ↑ 3 Changed 9 years ago by fake51

Replying to jeanyves:

My two changes are to test on addcommentpage and addcommend.column3

@Peter, I still see the bug in production. I will move my additional fix in Alpha where it is to be tested

Question why do we only see this bug now ?

Committed your fix to the template, seems to work. I'd like to see the logging work before it goes live

comment:6 Changed 7 years ago by planetcruiser

  • Resolution set to fixed
  • Status changed from new to closed

looks good to me now

comment:7 Changed 6 years ago by TimLoal

  • Milestone changed from unassigned to Legacy Resolved
Note: See TracTickets for help on using tickets.