[AppDB] Re: New slash Bug - and more ..

[Tony Lambregts]

Nick Law wrote:
>
>
> Nick Law wrote:
>>
>>
>> Nick Law wrote:
>>> Hi Tony,
>>>
>>>  I haven't had a chance to investigate the formatting issue yet, but 
>>> in the meantime I can confirm another bug.
>>>
>>> This bug corrupts posts as follows....
>>>
>>> If you post a message on an APPDB page  such as
>>>
>>> /*> sudo nice -n 10 wine "C:\Program Files\World of Warcraft\WoW.exe"*/
>>>
>>> and then post it, when you then look at the posts this is what you 
>>> get .....
>>>
>>> /*> sudo nice -n 10 wine "C:Program FilesWorld of WarcraftWoW.exe"*/
>>>
>>> _Notice that all the back slashes have been removed ....._
>>>
>>> however if you then reply to this message, you get the original 
>>> message displayed correctly, it displays it incorrectly when you 
>>> just view the posts. So the AppDB database ( MySQL ? ) is storing 
>>> the string correctly it's just incorrectly removing them in order to 
>>> display them but only in the view mode not the reply mode. This bug 
>>> is 100% reproducable.
>>>
>>> Hope that makes sense.
>>>
>>> There's an example of this at the bottom of  
>>> http://appdb.winehq.org/appview.php?versionId=5109 (post dated 
>>> 02/Jul/06 with the above sudo command in it.
>>>
>>> -- 
>>> Regards
>>> Nick Law
>>>
>>>   
>> Oh, I forgot to mention ....
>>
>> If you type ...
>> /*> sudo nice -n 10 wine "C:\\Program Files\\World of Warcraft\\WoW.exe"
>>
>> */i.e with two backslashes where there should be one then the text is 
>> displayed correctly, i.e with single backslashes.
>> Cheers
>> Nick
> Also if the post says something like ............ (this post has got 
> the slashes in because it was posted with two slashes)
> */
> > sudo nice -n 10 wine "C:\Program Files\World of Warcraft\WoW.exe"
>
> /*Then the email of this post that you get is the message above but 
> with two slashes ( the way it's stored in the database)
>
> I wonder if that means if you put a \n in a post you get a newline ? 
> Haven't tested that yet ..
>
> Have fun
> Nick
>
>
No \n is not an issue we just have a problem with slashes being remove 
as you can see from the link.
http://appdb.winehq.org/appview.php?versionId=5109&mode=nested#Comment-13192
I know what is going on here and it is not really a surprise.  I am 
CC:ing  the other developers with a patch.

Change Log: Remove unneeded stripslashes()

Files Changed: include/comment.php

-------------- next part --------------
A non-text attachment was scrubbed...
Name: FixCommentDisplay.diff
Type: text/x-patch
Size: 700 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20060701/702203d4/FixCommentDisplay.diff