#50 Wrong db table field for map votes in admin_mapvoting.php

0.97.3.5
open
Jarrod Mast
3
2006-01-03
2005-10-21
synth_spring
No

On line 38 of the admin_mapvoting.php file is the
following code:

$x->add_extra('del',array(array('DELETE FROM
poll_votes_maps','id')));

This should probably be changed to:

$x->add_extra('del',array(array('DELETE FROM
poll_votes_maps','mapid')));

When a map with a specific 'id' value is deleted, the
poll_votes_maps row(s) with the same 'mapid' value
should also be deleted. Map vote rows that just have
the same 'id' value as the map should not be deleted.

Discussion

  • Jarrod Mast
    Jarrod Mast
    2005-11-16

    • assigned_to: nobody --> skullshot
     
  • Jarrod Mast
    Jarrod Mast
    2005-11-20

    • labels: --> Other Modules
    • priority: 5 --> 3
    • status: open --> pending
     
    • status: pending --> closed
     
  • Logged In: YES
    user_id=1312539

    This Tracker item was closed automatically by the system. It was
    previously set to a Pending status, and the original submitter
    did not respond within 14 days (the time period specified by
    the administrator of this Tracker).

     
  • synth_spring
    synth_spring
    2006-01-03

    • status: closed --> open
     
  • Logged In: YES
    user_id=1379082

    Hey Skull, is the solution for this problem as easy as
    changing that line or is there more to this?