• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

I've attached a file style#1.css. Now FILENAME tells me this is file style, so some part is missing from the name.

The url parameters are: ?filename=style#1.css&revInfo=1

FILEPATH has the same problem, it says: http://visiblearea.com/devtwiki/pub/Sandbox/BigImage/style


Moving/renaming is also not possible, because param style is passed to that page and twiki thinks file style does not exist - which is right.


I just tried this on TWiki0400 branch, with a file called blah#de#blah.gif, and it works fine. DIscarding pending a clearer report (e.g. what are FILENAME and FILEPATH and where are they used?)


I have attached a file called test#1.txt to this topic. Check http://develop.twiki.org/~develop/cgi-bin/attach/Bugs/Item1724?filename=test#1.txt&revInfo=1. You will see that the page refers to file test.


Sounds like a simple problem with URL-encoding, perhaps? Normally the # suggests a label in the URL, probably the manage link (and others?) should just be URL-encoded to work correctly.

-- SP

Is it the locale, possibly? When I hover over Arthur's link above, I see the correct file; when I click on it, I get the file. I can't see any problem....


Well, you can get the latest revision of the file OK, but the revision table is empty at the manage screen (you can't get to previous versions).

I'm attaching a patch, but I'm a bit unweary of two things:

  1. Why doesn't TWiki::urlEncode encode the # - I'd say it should?
  2. Is this I18N-safe "enough" to be committed?

-- SP

Oh, I see. The problem is simply that the templates include an unencoded URL. All that's needed is to put %ENCODE around the %A_FILE in attachtables.tmpl. # is URL encoded just fine.

SVN 9039, 9040 (already merged)


*Sigh* Why didn't I think of that smile

Excellent, thanks!

-- SP

With the updated skin, this needs a second look.

  1. Direct link in "Update attachment .." in the header doesn't work (The requested URL was not found on this server)
  2. Move link doesn't work ("Attachment does not exist")
  3. Delete link doesn't work ("Attachment does not exist")

-- SP

Attachment filename seemed to have accidentially been removed from the forms, added again.

SVN 9090, 9091 + 9093.

-- SP

Merged to TWiki4, SVN 9124


9094 + 9125 ready for merge.

I believe svn log should be used in preference to the listed SVN revisions. "Hidden" commits like 9094 will probably cause some trouble later if never merged.

-- SP

The whole TWiki:Plugins.PatternSkin stuff is already merged, so I'm closing this one.

-- RA

Closed with the release of 4.0.2


Summary Attach: FILENAME AND FILEPATH not showing the file name with non-alpha characters
ReportedBy TWiki:Main.ArthurClemens

SVN Range Tue, 21 Feb 2006 build 8854
AppliesTo Engine

Priority Urgent
CurrentState Closed

Checkins 9039 9040 9090 9091 9093 9124 9125
TargetRelease patch
Topic attachments
I Attachment History Action Size Date Who Comment
Texttxt test#1.txt r1 manage 0.1 K 2006-03-02 - 13:13 UnknownUser Test file to check how the attach screens will behave on the filename
Unknown file formatpatch urlencode_filenames_in_links.patch r1 manage 5.6 K 2006-03-02 - 19:56 UnknownUser Patch suggestion for allowing # and other chars in filenames
Edit | Attach | Watch | Print version | History: r20 < r19 < r18 < r17 < r16 | Backlinks | Raw View |  Raw edit | More topic actions
Topic revision: r20 - 2006-04-01 - KennethLavrsen
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2023 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback