Recursive comment references possible

Found a bug? Tell us!!
Post Reply
urs.enke
Regular
Posts: 10
Joined: Sat Jun 09, 2007 1:02 am
Location: Hamburg, Germany

Recursive comment references possible

Post by urs.enke »

When editing a comment, it is possible to make it the child of one of its own offspring. It's stupid to do, but the involved comments then disappear until parenthood is set again.
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Recursive comment references possible

Post by garvinhicking »

Hi!

Good catch! Fixed in SVN. :-)

Best regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
urs.enke
Regular
Posts: 10
Joined: Sat Jun 09, 2007 1:02 am
Location: Hamburg, Germany

Re: Recursive comment references possible

Post by urs.enke »

It is now impossible to make an entry a child of itself, but one can still make it a child of a child, grandchild etc. With the current database structure, there will have to be a non-trivial query to determine all offspring. Alternatively, when the list of possible parents is created for the edit dialog, the branch could be cut that starts with the article being edited.
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Recursive comment references possible

Post by garvinhicking »

Hi!

I agree, that is still possible. But that lies in the responsibility of an Admin to NOT edit an entry to do that. :-)

In some cases it is wanted that you can change complete tree structures, so I don't think we should remove the option that people can freely attach structures.

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
urs.enke
Regular
Posts: 10
Joined: Sat Jun 09, 2007 1:02 am
Location: Hamburg, Germany

Re: Recursive comment references possible

Post by urs.enke »

In some cases it is wanted that you can change complete tree structures, so I don't think we should remove the option that people can freely attach structures.
Surely, I was only talking about cutting that branch of the tree that starts with the article being edited. This would leave all (sensible) options to the user.
Post Reply