http://urbangiraffe.com/support/ Urban Giraffe Support Topic: A bug, a fix and a suggestion en Fri, 21 Nov 2008 20:37:31 +0000 http://urbangiraffe.com/support/topic/a-bug-a-fix-and-a-suggestion#post-352 Fri, 22 Aug 2008 07:26:31 +0000 John 352@http://urbangiraffe.com/support/ <p>Thanks Nikola </p> http://urbangiraffe.com/support/topic/a-bug-a-fix-and-a-suggestion#post-164 Thu, 10 Apr 2008 12:29:44 +0000 weboholic 164@http://urbangiraffe.com/support/ <p>Hi,</p> <p>I've posted a bug as comment on the website, then I fixed it and wanted to post the solution and then I sow the links to the forum and the tracker :)</p> <p>Here is the bug:<br /> What are the chances that it's cont compatible with WP 2.5? When I try to create a new redirect, I get a success - the new row in the list is created. But when I check the DB - there's no entry made.</p> <p>Here is the fix:<br /> OK, I've fixed it. Here is the error msg:</p> <blockquote><p>Incorrect integer value: '' for column 'regex' at row 1</p> </blockquote> <p>which led me to the problem. It might not have been a WP 2.5 issue but with mysql 5.0.51a. I had to remove surrounding single quotes on $red-&gt;regex in the query on line 144, so it works now.</p> <p>And here is the suggestion I would like to make:<br /> However I would like to point you to a little design change that you might want to consider: the plugin is using an int data type column for storing a <em>boolean</em> value. This is a huge space waste. An INT data type reserves 4 bytes for each row, while a TINYINT only 1 byte and this is the smallest you can get. Well 75% improvement is pretty good. This will also speed up querying the table which is important if you have a lot of redirects like me :)</p> <p>I haven't studied the whole plugin, nor do I intend to, so this next remark might not fit at all. It just seems to me that the column position is redundant with the column id.</p> <p>This a live saving plugin.<br /> Greetings<br /> Nikola </p>