126821 - publish location ending in a '/' should not be trimmed

Status: VERIFIED FIXED

(SeaMonkey :: Composer, defect)

Description

[sujay]

publish location not ending in a '/'
it really should

a directory *always* ends in a slash (otherwise you couldn't tell if it was a 
filename without an extension or a directory)

Comment 1

[Charles Manske]

nope. In order to control the madness of always checking for "/" at the end 
of directories, we have these rules:
1. The "publish" and "browse" urls in the dialogs (and in prefs) never have the 
terminal "/"
2. Whenever we publish, we *always* append the "directory" you see in the 
Publish panel of the Publish dialog. That string *always* begins and ends with
"/" (it may simply be "/" for the "root" directory of a site)
When you publish, the full destination URL is:
publishUrl + /directory/ + filename
thus we are guarenteed to always have the proper "/" 

Comment 2

[Kathleen :Brade]

Reopen

Users expect publish locations to end in a slash.
In reality, if there isn't a slash, you really can't tell if it's a directory or
a file.  Our UI needs some smarts to always add a slash on the end.

I don't care about the prefs, but the UI needs to be consistent with what users
are told by their ISP or other hosting service.  Removing the slash will confuse
them.

As for your second point, I'll argue that the publish / browse locations should
end in a slash (as above) and the Directory edit field in more should NOT begin
with a slash.  A leading slash there will lead some people to think "root of the
file system" or root of the server (like a relative url).

I think we should require an ending slash in publish/browse location edit
fields.  The other directory publish fields should end with a slash (if there is
any data in them).

Comment 3

[Kathleen :Brade]

Daniel gave me this example of for publishing:
  http://www.myhost.com/helpers/publish.cgi.file=filename.htm

After further discussion with Daniel, I believe we should always and
automatically add a slash to the publish location when it doesn't end in one of
these characters:  & = ? /

Comment 4

[Charles Manske]

ok, whatever. Just wish we decided that before.

Comment 5

[Charles Manske]

brade, glazman: If we are allowing funky publish urls that end in "&", "=", 
or "?", do you know if a filename with a "subdirectory" is allowed with those? 
E.g., would  http://www.myhost.com/helpers/publish.cgi.file=myfiles/filename.htm
work?

Comment 6

[Charles Manske]

Fixed as part of patch to bug 88208

Comment 7

[michael wendell]

Verified.   On the 03-08 trunk build, I am getting the ending / in the publish 
location.

Comment 8

[sujay]

I also verified this is fixed in today's 3/8 trunk build.