Issue attachments not included in bitbucket_issue_converter.py

GoogleCodeExporter commented 9 years ago

I plan to migrate a project from GoogleCode to Bitbucket. The main reason not 
choosing GitHub was it's issue handler doesn't support file attachments(just 
pictures). But Bitbucket does! The GoogleCodeProjectHosting.json file created 
by Takeout contains all necessary info to concatenate an attachment url ,so it 
would be nice to have a command line option to migrate file attachments of 
issue comments too.

Original issue reported on code.google.com by gbtami on 24 Jan 2015 at 8:34

GoogleCodeExporter commented 9 years ago

Created a clone https://code.google.com/r/gbtami-googlecode2bitbucket/
It converts attachments and updates to logs. (The code contains some PyChess 
specific labels and milestones.)

Original comment by gbtami on 27 Jan 2015 at 8:37

GoogleCodeExporter commented 9 years ago

Attached is a patch (download_support.patch) created at revision 8ac29fcfa 
based on the changes from comment #1 and including the following enhancements:

- Fix the project download URL (all project URLs have the form 
https://code.google.com/p/...).
- Don't re-download an attachment if it already exists in the attachments path.
- Also unescape "&" in FixUpComment.
- Remove PyChess-specific logic.

Also attached is the example of converting an issue from Google Code to 
Bitbucket:
example_google_code.pdf = Original issue.
example_bitbucket.pdf = Bitbucket import including the changes from this patch.

One additional nice-to-have feature would be better formatting [1] in 
FixUpComment for comments that include code snippets or call stacks. The 
default Markdown formatting for these is usually quite bad, including breaking 
other Markdown syntax if the comment code brackets (`{}`) are unmatched (see 
comments #1 and #4 in the example pdf for examples of bad formatting).

Also, it should be noted that the target attachments folder (passed via 
--attachments_path=<path>) needs to be relative to the db-1.0.json file 
location and included in the zip archive uploaded in Bitbucket.

[1] 
https://bitbucket.org/tutorials/markdowndemo/overview#markdown-header-code-and-s
yntax-highlighting

Original comment by magreenb...@gmail.com on 13 Mar 2015 at 10:54

Attachments:

GoogleCodeExporter commented 9 years ago

Attached is an updated patch with a few more minor enhancements:

- Add a space after "@" to avoid user references.
- Escape "\", "`", "_", "*" and "#" which are otherwise treated as formatting. 
This makes python and C++ call stacks look decent.
- Map "done" and "verified" statuses to "resolved".

Original comment by magreenb...@gmail.com on 14 Mar 2015 at 2:48

Attachments:

download_support2.patch

GoogleCodeExporter commented 9 years ago

@magreenblatt, thank you for the patches. I'll try to merge them in as soon as 
I can.

Original comment by chrsm...@google.com on 14 Mar 2015 at 2:58

GoogleCodeExporter commented 9 years ago

Thank for the patch. However, I get a 404 error when trying to download first 
attachement with gbtami cloned script and a "Failed to download..." message 
error with the above patch for all attachments. When directly  typing the 
address returned by the error log in a browser, the file is however accessible.

Could you tell me what I'm doing wrong and if I'm missing something to allow 
HTTP downloads from a python script (windows user here) ?

Original comment by ekeeke31@gmail.com on 15 Mar 2015 at 6:18

GoogleCodeExporter commented 9 years ago

@comment#5: You'll get the "Failed to download..." error with the patch if the 
directory passed to `--attachments_path` does not already exist.

Original comment by magreenb...@gmail.com on 15 Mar 2015 at 7:02

GoogleCodeExporter commented 9 years ago

Thanks for your answer.

The directory passed as parameter exists but it seems like the patch actually 
fails if the attachment file does not exist locally.

I fixed it by removing the following line:
if not os.path.exists(path):

At this step, path contains the full pathname for the file (attachments_path/ 
attachmentId) so I'm not sure if this intended behavior or a bug.

Original comment by ekeeke31@gmail.com on 15 Mar 2015 at 7:28

GoogleCodeExporter commented 9 years ago

In order to migrate attachment to github, you can use the attached patch

However, inlining images doesn't work on GitHub right now - already reported 
there.

Original comment by sstrickr...@googlemail.com on 11 Apr 2015 at 6:47

Attachments:

0001-Handle-attachments.patch

GoogleCodeExporter commented 9 years ago

Submitted changes to fix this to GitHub (thanks @sstrickroth). The changes for 
Bitbucket haven't been merged into the main branch, though I assume patching 
the two attached files would work to unblock users.

Original comment by chrsm...@google.com on 24 Apr 2015 at 8:52

Changed title: Issue attachments not included in bitbucket_issue_converter.py
Changed state: Accepted
Added labels: Priority-Low, Service-Bitbucket
Removed labels: Priority-Medium

GoogleCodeExporter commented 9 years ago

<a href="http://www.alrayyanclean.com/index.html">شركة تنظيف 
بالرياض</a>
<a href="http://www.alrayyanclean.com/index.html">شركة نظافة 
بالرياض</a>
<a href="http://www.alrayyanclean.com/index.html">شركات تنظيف 
بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D9%86%D
8%A7%D8%B2%D9%84.html">شركة تنظيف منازل بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D9%86%D
8%A7%D8%B2%D9%84.html">شركة نظافة منازل بالرياض</a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D9%86%D
8%A7%D8%B2%D9%84.html">شركات تنظيف منازل بالرياض</a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D9%86%D
8%A7%D8%B2%D9%84.html">شركة تنظيف شقق بالرياض</a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D9%86%D
8%A7%D8%B2%D9%84.html">شركة نظافة شقق بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D9%86%D
8%A7%D8%B2%D9%84.html">شركات تنظيف شقق بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%81%D9%84%D
9%84.html">شركة تنظيف فلل بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%81%D9%84%D
9%84.html">شركات تنظيف فلل بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D8%AE%D8%B2%D
8%A7%D9%86%D8%A7%D8%AA.html">شركة تنظيف خزانات بالرياض 
</a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D8%AE%D8%B2%D
8%A7%D9%86%D8%A7%D8%AA.html">شركات تنظيف خزانات بالرياض 
</a>
<a 
href="http://www.alrayyanclean.com/%D8%B1%D8%B4_%D9%85%D8%A8%D9%8A%D8%AF%D8%A7%D
8%AA.html">شركة رش مبيدات بالرياض</a>
<a 
href="http://www.alrayyanclean.com/%D8%B1%D8%B4_%D9%85%D8%A8%D9%8A%D8%AF%D8%A7%D
8%AA.html">شركات رش مبيدات بالرياض</a>
<a 
href="http://www.alrayyanclean.com/%D8%B1%D8%B4_%D9%85%D8%A8%D9%8A%D8%AF%D8%A7%D
8%AA.html">شركة مكافحة حشرات بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%B1%D8%B4_%D9%85%D8%A8%D9%8A%D8%AF%D8%A7%D
8%AA.html">شركات مكافحة الحشرات بالرياض</a>
<a 
href="http://www.alrayyanclean.com/%D9%83%D8%B4%D9%81_%D8%AA%D8%B3%D8%B1%D8%A8%D
8%A7%D8%AA.html">شركة كشف تسربات المياه بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AA%D9%86%D8%B8%D9%8A%D9%81_%D9%85%D8%AC%D
8%A7%D9%84%D8%B3.html">شركة تنظيف مجالس بالرياض </a>
<a 
href="http://www.alrayyanclean.com/%D8%AC%D9%84%D9%89_%D8%A8%D9%84%D8%A7%D8%B7.h
tml">شركة جلى بلاط بالرياض </a>

Original comment by alrayyan...@gmail.com on 14 Sep 2015 at 9:17

dina890 commented 9 years ago

http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D9%83%D8%B4%D9%81-%D8%AA%D8%B3%D8%B1%D8%A8%D8%A7%D8%AA-%D8%A7%D9%84%D9%85%D9%8A%D8%A7%D9%87-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/

dina890 commented 9 years ago

http://b-yout.com/

dina890 commented 9 years ago

[url=http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D9%83%D8%B4%D9%81-%D8%AA%D8%B3%D8%B1%D8%A8%D8%A7%D8%AA-%D8%A7%D9%84%D9%85%D9%8A%D8%A7%D9%87-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/]افضل شركة كشف تسربات المياه بالرياض والدمام[/url] [url=http://b-yout.com/]شركة البيوت[/url] , [url=http://b-yout.com/%D8%A3%D9%81%D8%B6%D9%84-%D8%B4%D8%B1%D9%83%D8%A9-%D9%86%D9%82%D9%84-%D8%B9%D9%81%D8%B4-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/]نقل عفش[/url] و[url=http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D8%AA%D9%86%D8%B8%D9%8A%D9%81-%D9%83%D9%86%D8%A8-%D8%A8%D8%AC%D8%AF%D8%A9/]تنظيف المنا[/url]زل وجلى البلاط وكشف تسربات المياه و كشف [url=http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D9%83%D8%B4%D9%81-%D8%AA%D8%B3%D8%B1%D8%A8%D8%A7%D8%AA-%D8%A7%D9%84%D9%85%D9%8A%D8%A7%D9%87-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/]تسربات الغاز[/url] و[url=http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D8%B1%D8%B4-%D9%85%D8%A8%D9%8A%D8%AF%D8%A7%D8%AA-%D9%88%D9%85%D9%83%D8%A7%D9%81%D8%AD%D8%A9-%D8%A7%D9%84%D8%AD%D8%B4%D8%B1%D8%A7%D8%AA/]رش المبيدات ومكافحة[/url] الحشرات_والترميمات_وعزل مائىللأسطح والخزانات وحمامات السباحة وتستخدم الشركة اساليب متطورة من عملية تطوير الذات و تحسين الكفاءة لتقدم خدمة كاملة الي عملائها و تحصل على اكبر قدر من التميز بافضل جودة وافضل اسعار لاشك ان المنافسة قوية ولكن توجد فرصة كبيرة للتميز سنصلك اينما كنت فنحن نعمل على مدار الاسبوع . نحن الأفضل للأننا نخدم الافضل [url=http://b-yout.com/]شركة البيوت للخدمات المنزل[/url]ية [url=http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D9%83%D8%B4%D9%81-%D8%AA%D8%B3%D8%B1%D8%A8%D8%A7%D8%AA-%D8%A7%D9%84%D9%85%D9%8A%D8%A7%D9%87-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/]افضل شركة لكشف تسربات المياه بالر[/url]ياض و افضل شركة نقل اثاث بالرياض ومتخصصون في كش[url=http://b-yout.com/%D8%B4%D8%B1%D9%83%D8%A9-%D9%83%D8%B4%D9%81-%D8%AA%D8%B3%D8%B1%D8%A8%D8%A7%D8%AA-%D8%A7%D9%84%D9%85%D9%8A%D8%A7%D9%87-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/]ف تسربات المياه في الرياض[/url]

dina890 commented 9 years ago

dina890 commented 9 years ago

dina890 commented 9 years ago

http://b-yout.com/%D8%A7%D9%81%D8%B6%D9%84-%D8%B4%D8%B1%D9%83%D8%A9-%D8%AA%D8%AE%D8%B2%D9%8A%D9%86-%D8%A7%D8%AB%D8%A7%D8%AB-%D8%A8%D8%A7%D9%84%D8%B1%D9%8A%D8%A7%D8%B6/

ekhtsasy commented 8 years ago

فقط لدى شركة تنظيف خزانات بالرياض افضل المواد المستخدمة فى تنظيف خزانات المياه وافضل الامكانيات المادية والبشرية التى تقدم افضل نتائج التنظيف على الاطلاق حيث تعتمد الشركة على افضل عماله فليبنية متخصصه فى غسيل الخزانات من الداخل بمهاره وكفاءة عاليه وبأسعار رمزيه فاسرعو بالتعاقد معنا واتصلو بنا من اى مكان بالسعودية نكون فى خدمتكم شركة تنظيف منازل بالرياض تنظيف منازل شركة تنظيف خزانات بالرياض نقدم لكم افضل خدمات نقل الاثاث داخل وخارج الرياض مع خدمات التغليف المجانية وذلك لضمان نقل الاثاث بدون كسر أو خدوش وذلك من خلال شركتنا لنقل الاثاث بالرياض والتى تتميز بالجودة العاليه والكفاؤة المطلقة فى التنفيذ والمصداقيه الكبيرة فى الاتفاق وموعد التسليم , فشركتنا تتميز بالتزامها الصارم بكافة اتفاقاتها مع العميل , فاتصلو بنا من اى مكان بالرياض لنرسل لكم منوبينا ليكونو فى خدمتكم شركة نقل اثاث بالرياض شركة نقل عفش بالرياض

hmariwa / support-tools

Issue attachments not included in bitbucket_issue_converter.py #10