[Zope] CVSFile - can´t update file in sandbox

[Craeg K Strong]

This is a multi-part message in MIME format.
--------------030307030506030706000001
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Hello:

You have found a bug!      Sorry about that.   It is a bit of sloppiness 
on my part--
I did not test that exact use case.   Anyway, attached please find a 
"diff -u" file
that illustrates the (small) change you must make to the file
ExternalFile.py to fix the bug.    I will be making another release, but 
this way you
can get the fix going immediately.  

Let me know if you need help applying the patch (not sure if you have 
had experience dealing
with patchfiles before).   Again, the file to patch is ExternalFile.py 
which is part of the
ExternalFile 0.9.0 product (which CVSFile 0.7 inherits from).  

After you apply the patch, you can restart your Zope server and all 
should be well.
Let me know how it works out for you.

Regards,

--Craeg Strong

Oliver Skiebe wrote:

>hi there,
>
>i am running CVSFile 0.7 on Zope 2.5.0;
>
>when i try to update a checked out file in the sandbox with a local file
>(i.e. upload that file), zope keeps telling ma
>
>"... Invalid request
>The parameter, externalfile_content, was omitted from the request...."
>
>This only happens when uploading binary files - e.g. Images -, text files
>work just normal.
>Creating new binary files and committing them to cvs works as well...
>
>Any suggestions out there? - Thank you!
>
>- Oliver
> 
>_____
>To understand recursion, we first have to understand recursion.
> 
>
>
>
>_______________________________________________
>Zope maillist  -  Zope@zope.org
>http://lists.zope.org/mailman/listinfo/zope
>**   No cross posts or HTML encoding!  **
>(Related lists - 
> http://lists.zope.org/mailman/listinfo/zope-announce
> http://lists.zope.org/mailman/listinfo/zope-dev )
>



--------------030307030506030706000001
Content-Type: application/x-java-vm;
 name="externalfile-changes"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
 filename="externalfile-changes"

SW5kZXg6IEV4dGVybmFsRmlsZS5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvcmVwb3Np
dG9yeS9jdnNyb290L2NvbS9hcmllbHBhcnRuZXJzL3pvcGUvRXh0ZXJuYWxGaWxlL0V4dGVy
bmFsRmlsZS5weSx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS42MQpyZXRyaWV2aW5nIHJldmlz
aW9uIDEuNjIKZGlmZiAtdSAtcjEuNjEgLXIxLjYyCi0tLSBFeHRlcm5hbEZpbGUucHkJMjAw
Mi8wMy8xMyAxOTo0Mjo1MwkxLjYxCisrKyBFeHRlcm5hbEZpbGUucHkJMjAwMi8wNC8wMSAw
MToxMzoxNAkxLjYyCkBAIC05LDggKzksOCBAQAogIiIiCiAKIF9fY3ZzdGFnX18gID0gJyRO
YW1lOiAgJCdbNjotMl0KLV9fZGF0ZV9fICAgID0gJyREYXRlOiAyMDAyLzAzLzEzIDE5OjQy
OjUzICQnWzY6LTJdCi1fX3ZlcnNpb25fXyA9ICckUmV2aXNpb246IDEuNjEgJCdbMTA6LTJd
CitfX2RhdGVfXyAgICA9ICckRGF0ZTogMjAwMi8wNC8wMSAwMToxMzoxNCAkJ1s2Oi0yXQor
X192ZXJzaW9uX18gPSAnJFJldmlzaW9uOiAxLjYyICQnWzEwOi0yXQogCiAjCiAjIEFkZCBv
dXIgYmFzZSBjbGFzc2VzLiAgRGVyaXZlZCBjbGFzc2VzIHdpbGwgZ2V0IHRoaXMgc3R1ZmYg
YXV0b21hdGljYWxseQpAQCAtNjgyLDcgKzY4Miw4IEBACiAgICAgX21pblJvd3MgPSA1CiAK
ICAgICBfc2VjdXJpdHkuZGVjbGFyZVByb3RlY3RlZCgnRWRpdCcsICdtYW5hZ2VfZWRpdEFj
dGlvbicpCi0gICAgZGVmIG1hbmFnZV9lZGl0QWN0aW9uKHNlbGYsIGV4dGVybmFsZmlsZV9j
b250ZW50LCBTVUJNSVQsCisgICAgZGVmIG1hbmFnZV9lZGl0QWN0aW9uKHNlbGYsIGV4dGVy
bmFsZmlsZV9jb250ZW50ID0gTm9uZSwKKyAgICAgICAgICAgICAgICAgICAgICAgICAgU1VC
TUlUID0gJ0NhbmNlbCBDaGFuZ2VzJywKICAgICAgICAgICAgICAgICAgICAgICAgICAgZWRp
dF90ZXh0YXJlYV9jb2xzID0gJzgwJywKICAgICAgICAgICAgICAgICAgICAgICAgICAgZWRp
dF90ZXh0YXJlYV9yb3dzID0gJzQwJywKICAgICAgICAgICAgICAgICAgICAgICAgICAgdXBs
b2FkX2ZpbGUgPSBOb25lLApAQCAtNzAyLDcgKzcwMyw3IEBACiAKICAgICAgICAgZWxpZiBT
VUJNSVQgPT0gJ0NhbmNlbCBDaGFuZ2VzJzoKICAgICAgICAgICAgIG1lc3NhZ2UgPSAnQ29u
dGVudCBjaGFuZ2VzIGNhbmNlbGxlZC4nCi0gICAgICAgICAgICByZXR1cm4gc2VsZi5tYW5h
Z2VfZWRpdEZvcm0oZXh0ZXJuYWxmaWxlX2NvbnRlbnQ9Tm9uZSwKKyAgICAgICAgICAgIHJl
dHVybiBzZWxmLm1hbmFnZV9lZGl0Rm9ybShleHRlcm5hbGZpbGVfY29udGVudCwKICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBoZWFkZXJfbWVzc2FnZT1tZXNz
YWdlKQogCiAgICAgICAgIGVsaWYgU1VCTUlUID09ICdVcGxvYWQgRmlsZSc6CkBAIC03MTks
NyArNzIwLDcgQEAKICAgICAgICAgICAgIGlmIHZhbGlkOgogICAgICAgICAgICAgICAgIHNl
bGYuc2V0Q29udGVudHModXBsb2FkX2NvbnRlbnRzLCBSRVFVRVNUKQogICAgICAgICAgICAg
ICAgIG1lc3NhZ2UgPSAnQ29udGVudCBjaGFuZ2VkLiAoJXMpJyAlIERhdGVUaW1lKCkuSVNP
KCkKLSAgICAgICAgICAgICAgICByZXR1cm4gc2VsZi5tYW5hZ2VfZWRpdEZvcm0oZXh0ZXJu
YWxmaWxlX2NvbnRlbnQ9Tm9uZSwKKyAgICAgICAgICAgICAgICByZXR1cm4gc2VsZi5tYW5h
Z2VfZWRpdEZvcm0oZXh0ZXJuYWxmaWxlX2NvbnRlbnQsCiAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgIGhlYWRlcl9tZXNzYWdlPW1lc3NhZ2UpCiAgICAg
ICAgICAgICBlbHNlOgogICAgICAgICAgICAgICAgIG1lc3NhZ2UucmVwbGFjZSgnXG4nLCc8
YnIvPicpCg==
--------------030307030506030706000001--