23 Apr
2010
23 Apr
'10
3:32 a.m.
On 04/06/2010 05:03 PM, Vincent Massol wrote:
Hi Milind,
On Apr 6, 2010, at 5:00 PM, Milind Kamble wrote:
This is the issue that started it: XWIKI-2087
So, there were three main problems:
1. Impossible to actually restore the attachment from the database since
the ID was generated using the hash of the original, correct name, yet
it was stored using the broken name, with ? instead of non-latin1 characters
2. Impossible to link to such an attachment, since a non-UTF wiki would
encode non-ASCII chars to their &#xyz; escapes, and the filename wasn't
decoded when trying to get the attachment from the database
3. Encoding bug in the old WYSIWYG which composed the URL using a wrong
encoding
3 should be fixed since we're forcing UTF-8 in URLs.
2 and 1 should work if the wiki+database are using UTF8, but they might
still fail in latin1.
Denis,
I understand your point that XE being used globally, needs to support more than Ascii
char set.
While the new reference model matures, could you clarify if underscore in a file name
would break the functionality under the current model where attachment name is used as a
reference for attachments? If not, would it be possible to eliminate the stripping of just
the underscore chars and push that fix in the next XE release -- I am OK with space chars
getting stripped off.
I don't think that underscores are a problem even with the old "reference as
string" code. Actually I don't even know why we're stripping them. Sergiu
might know more. Any idea Sergiu? Thanks
-Vincent
> ________________________________
> From: Denis Gervalle<dgl(a)softec.lu>
> To: XWiki Developers<devs(a)xwiki.org>
> Sent: Tue, April 6, 2010 8:30:34 AM
> Subject: Re: [xwiki-devs] Simple patch to enable/preserve underscore chars in
attachment file names
>
> On Tue, Apr 6, 2010 at 14:02, Guillaume Lerouge<guillaume(a)xwiki.com> wrote:
>
>> Hi Milind,
>>
>> On Tue, Apr 6, 2010 at 1:23 AM, Milind Kamble<mbkads(a)yahoo.com> wrote:
>>
>>> Hi. I would like the dev community to evaluate this simple fix that will
>>> enable uploading of files with underscore chars in the file name when
>> users
>>> perform the attach action. Our user community is quite impressed about
>> the
>>> refreshing ease of use and the power, flexibility in their collaboration
>>> work flow made possible by XE. They would like to escape the tyranny of
>>> Microsoft-MOSS as early as possible and the main roadblock to do so is
>> the
>>> stripping of space and underscores from file names which were created in
>> a
>>> MS-Office centric environment.
>>>
>>
>> I can't do much about your underscore problem (though I promise I'll
poke
>> the developer sitting right next to me so that he looks at it).
>>
>
> I was already aware of this issue, and I have had similar problemqs with
> attachment, not only with "_", but also with accentuated chars etc...
> Restriction on attachment names will be easier to be changed when the new
> model model using references will be fully in
place, since attachment names
> are currently used as reference for attachments. Be sure I will take care to
> have it improve.
--
Sergiu Dumitriu
http://purl.org/net/sergiu/