http://codereview.appspot.com/955/diff/1/3 File codereview/models.py (right): http://codereview.appspot.com/955/diff/1/3#newcode65 Line 65: base = db.StringProperty() Why do you need that? ...
I should mention that this code should be treated as only prototype. That is the
reason I am blandly returning an info message directly from view.py (instead of
having a template etc).
> Are you asking for feedback? This code seems to be too
> incomplete to be able to comment much.
Sorry. I merely intended this as a starting point for Ondrej or who ever else
wanting to implement Plan B (as described on codereview-discuss list). I did
test it with "-l" option but not with regular case and now realize that setting
"base" to "None" in Issue is wrong.
I will try to do a more thorough job in future.
> Sorry. I merely intended this as a starting point for Ondrej > or who ever
else
> wanting to implement Plan B (as described on
> codereview-discuss list).
Thanks for your work! I based my upload base files patch on yours here:
http://codereview.appspot.com/1026
Issue 955: Allow upload with no base
(Closed)
Created 18 years ago by Raghu
Modified 16 years, 10 months ago
Reviewers: GvR, ondrej.certik
Base URL: http://rietveld.googlecode.com/svn/trunk/
Comments: 5