summary refs log tree commit diff stats
path: root/includes/hits.php
diff options
context:
space:
mode:
authorStarla Insigna <hatkirby@fourisland.com>2008-12-19 21:06:16 -0500
committerStarla Insigna <hatkirby@fourisland.com>2008-12-19 21:06:16 -0500
commit31ac283c5bae32c91629fa36adf71572597f2cd5 (patch)
tree442d8a466e70a34caf4547c34ca1704f90d9c9f4 /includes/hits.php
parent7c7a1832c679729e7e97657be90a6d5de0cd37db (diff)
downloadfourisland-31ac283c5bae32c91629fa36adf71572597f2cd5.tar.gz
fourisland-31ac283c5bae32c91629fa36adf71572597f2cd5.tar.bz2
fourisland-31ac283c5bae32c91629fa36adf71572597f2cd5.zip
Fixed Admin's movePending post deletion bug
Certain posts, when moved around, were strangely deleted from the pending queue. This was actually two seperate bugs, both causing the same problem.

1. When looking for the post to swap with, the movePending command would search for the next post with an ID greater than or less than the current
   ID, but it wouldn't actually sort the results correctly so that the corrent posts wouldn't neccessarily always be the post shown. This resulted in
   a seemingly random pending posts being deleted. This has been fixed by adding a simply "ORDER BY" clause to the SQL "SELECT" commands.

2. When re-inserting the pending posts into the queue (after swapping IDs), if one of the posts contained invalid characters requiring escaping,
   MySQL would reject the post without error and simply not insert it, resulting in one or more of the posts involved in the switch to be deleted.
   This has been fixed by wrapping the text of the post in the mysql_real_escape_string() function.
Diffstat (limited to 'includes/hits.php')
0 files changed, 0 insertions, 0 deletions