diff options
author | Starla Insigna <hatkirby@fourisland.com> | 2008-12-19 21:06:16 -0500 |
---|---|---|
committer | Starla Insigna <hatkirby@fourisland.com> | 2008-12-19 21:06:16 -0500 |
commit | 31ac283c5bae32c91629fa36adf71572597f2cd5 (patch) | |
tree | 442d8a466e70a34caf4547c34ca1704f90d9c9f4 /theme | |
parent | 7c7a1832c679729e7e97657be90a6d5de0cd37db (diff) | |
download | fourisland-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 'theme')
0 files changed, 0 insertions, 0 deletions