Now nameless items can also be downloaded

Now nameless items can also be downloaded

[Thomas.J.Waldmann, 2014-06-05 21:02:50]

looks ok. maybe needs a bit "optimizing for filename usage" though.

[sksaurabhkathpalia, 2014-06-06 11:15:25]

On 2014/06/05 21:02:50, Thomas.J.Waldmann wrote:
> looks ok. maybe needs a bit "optimizing for filename usage" though.

What sort of optimization needs to be done?

[Thomas.J.Waldmann, 2014-06-14 17:40:40]

On 2014/06/06 11:15:25, sksaurabhkathpalia wrote:
> On 2014/06/05 21:02:50, Thomas.J.Waldmann wrote:
> > looks ok. maybe needs a bit "optimizing for filename usage" though.
> 
> What sort of optimization needs to be done?

well, usually one should avoid spaces and other special characters in filenames
(I guess on windows ":" could also be troublesome).

[Thomas.J.Waldmann, 2014-06-14 17:41:28]

> well, usually one should avoid spaces and other special characters in
filenames
> (I guess on windows ":" could also be troublesome).

so, for that, maybe add a filename method additionally to __unicode__.

[sksaurabhkathpalia, 2014-06-14 17:42:07]

On 2014/06/14 17:40:40, Thomas.J.Waldmann wrote:
> On 2014/06/06 11:15:25, sksaurabhkathpalia wrote:
> > On 2014/06/05 21:02:50, Thomas.J.Waldmann wrote:
> > > looks ok. maybe needs a bit "optimizing for filename usage" though.
> > 
> > What sort of optimization needs to be done?
> 
> well, usually one should avoid spaces and other special characters in
filenames
> (I guess on windows ":" could also be troublesome).

May be then we can check if name is there then filename should be name of item
else we can have itemid as name of file

[sksaurabhkathpalia, 2014-06-14 18:01:16]

On 2014/06/14 17:42:07, sksaurabhkathpalia wrote:
> On 2014/06/14 17:40:40, Thomas.J.Waldmann wrote:
> > On 2014/06/06 11:15:25, sksaurabhkathpalia wrote:
> > > On 2014/06/05 21:02:50, Thomas.J.Waldmann wrote:
> > > > looks ok. maybe needs a bit "optimizing for filename usage" though.
> > > 
> > > What sort of optimization needs to be done?
> > 
> > well, usually one should avoid spaces and other special characters in
> filenames
> > (I guess on windows ":" could also be troublesome).
> 
> May be then we can check if name is there then filename should be name of item
> else we can have itemid as name of file

I mean the new filename method would return itemid if name is not there and else
it returns name

[Thomas.J.Waldmann, 2014-07-05 15:56:39]

https://codereview.appspot.com/102210043/diff/20001/MoinMoin/util/interwiki.py
File MoinMoin/util/interwiki.py (right):

https://codereview.appspot.com/102210043/diff/20001/MoinMoin/util/interwiki.p...
MoinMoin/util/interwiki.py:112: returns the name of the file that is used for
downloading items
returns the filename that is used for downloading items