You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by jo...@apache.org on 2013/11/18 22:56:22 UTC
[1/6] git commit: [#6889] use .text() instead of .html() to avoid
html insertion of end-user input
Updated Branches:
refs/heads/cj/6837 0cbcbcf34 -> 5ecdd28fc (forced update)
[#6889] use .text() instead of .html() to avoid html insertion of end-user input
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/3a82e9de
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/3a82e9de
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/3a82e9de
Branch: refs/heads/cj/6837
Commit: 3a82e9de57a45d54888a0b84389a308fbc266887
Parents: 2f8d344
Author: Dave Brondsema <db...@slashdotmedia.com>
Authored: Mon Nov 18 15:32:41 2013 +0000
Committer: Dave Brondsema <db...@slashdotmedia.com>
Committed: Mon Nov 18 15:32:41 2013 +0000
----------------------------------------------------------------------
Allura/allura/lib/widgets/forms.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/3a82e9de/Allura/allura/lib/widgets/forms.py
----------------------------------------------------------------------
diff --git a/Allura/allura/lib/widgets/forms.py b/Allura/allura/lib/widgets/forms.py
index bf96445..4e1dae4 100644
--- a/Allura/allura/lib/widgets/forms.py
+++ b/Allura/allura/lib/widgets/forms.py
@@ -920,7 +920,7 @@ class NeighborhoodAddProjectForm(ForgeForm):
};
$.getJSON('suggest_name', data, function(result){
$unixname_input.val(result.suggested_name);
- $url_fragment.html(result.suggested_name);
+ $url_fragment.text(result.suggested_name);
check_names();
});
} else {
@@ -932,7 +932,7 @@ class NeighborhoodAddProjectForm(ForgeForm):
manual = true;
});
$unixname_input.keyup(function(){
- $url_fragment.html($unixname_input.val());
+ $url_fragment.text($unixname_input.val());
delay(check_names, 500);
});
});
[2/6] git commit: Update doc links
Posted by jo...@apache.org.
Update doc links
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/1280f025
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/1280f025
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/1280f025
Branch: refs/heads/cj/6837
Commit: 1280f025161fd75c725fea913c8934d91617b574
Parents: 3a82e9d
Author: Dave Brondsema <db...@slashdotmedia.com>
Authored: Fri Nov 15 18:33:53 2013 -0500
Committer: Dave Brondsema <db...@slashdotmedia.com>
Committed: Mon Nov 18 10:55:29 2013 -0500
----------------------------------------------------------------------
INSTALL.markdown | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/1280f025/INSTALL.markdown
----------------------------------------------------------------------
diff --git a/INSTALL.markdown b/INSTALL.markdown
index ed3698f..000ce36 100644
--- a/INSTALL.markdown
+++ b/INSTALL.markdown
@@ -146,8 +146,8 @@ register a new project in your own forge, visit /p/add_project
## Extra
-* Read more documentation: <http://allura.sourceforge.net/>
- * Including how to enable extra features: <http://allura.sourceforge.net/installation.html>
+* Read more documentation: <http://allura.sourceforge.net/docs/>
+ * Including how to enable extra features: <http://allura.sourceforge.net/docs/installation.html>
* Run the test suite (slow): `$ ALLURA_VALIDATION=none ./run_tests`
* File bug reports at <https://sourceforge.net/p/allura/tickets/new/> (login required)
* Contribute code according to this guide: <https://forge-allura.apache.org/p/allura/wiki/Contributing%20Code/>
[6/6] git commit: [#6837] Strip trailing question mark from "missing
wiki" links
Posted by jo...@apache.org.
[#6837] Strip trailing question mark from "missing wiki" links
Signed-off-by: Cory Johns <cj...@slashdotmedia.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/5ecdd28f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/5ecdd28f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/5ecdd28f
Branch: refs/heads/cj/6837
Commit: 5ecdd28fccfb85b8b2315e5fc7a12c321eddff27
Parents: 50d8074
Author: Cory Johns <cj...@slashdotmedia.com>
Authored: Mon Nov 18 21:56:08 2013 +0000
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Mon Nov 18 21:56:08 2013 +0000
----------------------------------------------------------------------
Allura/allura/scripts/trac_export.py | 5 +++++
ForgeImporters/forgeimporters/trac/tests/data/test-list.html | 2 +-
ForgeImporters/forgeimporters/trac/tests/test_tickets.py | 1 +
3 files changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/5ecdd28f/Allura/allura/scripts/trac_export.py
----------------------------------------------------------------------
diff --git a/Allura/allura/scripts/trac_export.py b/Allura/allura/scripts/trac_export.py
index 9377df3..0ca4b39 100644
--- a/Allura/allura/scripts/trac_export.py
+++ b/Allura/allura/scripts/trac_export.py
@@ -152,6 +152,7 @@ class TracExport(object):
url = self.full_url(self.TICKET_URL % id)
self.log_url(url)
d = BeautifulSoup(urlopen(url))
+ self.clean_missing_wiki_links(d)
desc = d.find('div', 'description').find('div', 'searchable')
ticket['description'] = html2text.html2text(desc.renderContents('utf8').decode('utf8')) if desc else ''
comments = []
@@ -269,6 +270,10 @@ class TracExport(object):
break
return self.get_ticket(id, extra)
+ def clean_missing_wiki_links(self, doc):
+ for link in doc.findAll('a', 'missing wiki'):
+ link.string = link.string.rstrip('?')
+
class DateJSONEncoder(json.JSONEncoder):
def default(self, obj):
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/5ecdd28f/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tests/data/test-list.html b/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
index 9c76868..2e96702 100644
--- a/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
+++ b/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
@@ -209,7 +209,7 @@ To reproduce:<br />
- main view closes, but "Scan view" is not cleared<br />
</p>
<p>
-By itself, this is not a big problem. However, if the "Identification view" was used before closing the main file, switching from "Scan view" to "Identification view" in the "Views" window will crash TOPPView.<br />
+By itself, this is not a big problem. However, if the "Identification view" was used before closing the main file, switching from "Scan view" to "Identification view" in the "Views" window will crash <a class="missing wiki">TOPPView?</a>.<br />
</p>
</div>
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/5ecdd28f/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tests/test_tickets.py b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
index c92b028..6d369e8 100644
--- a/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
+++ b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
@@ -241,3 +241,4 @@ class TestTracImportSupportFunctional(TestRestApiBase, TestCase):
ticket_num=390)
self.assertIn('To reproduce: \n\\- open an mzML file', ticket.description)
self.assertIn('duplicate of: \n\\- [#316](316)', ticket.discussion_thread.find_posts()[0].text)
+ self.assertIn('will crash TOPPView.', ticket.description)
[5/6] git commit: [#6837] Translate rev links when importing from Trac
Posted by jo...@apache.org.
[#6837] Translate rev links when importing from Trac
Also fixed a couple of issues with handling of ticket links and
escaping brackets in link text
Signed-off-by: Cory Johns <cj...@slashdotmedia.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/50d80744
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/50d80744
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/50d80744
Branch: refs/heads/cj/6837
Commit: 50d8074481aa7aa6dfe28d3ce30b6be526841403
Parents: e3b8453
Author: Cory Johns <cj...@slashdotmedia.com>
Authored: Fri Nov 15 22:39:26 2013 +0000
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Mon Nov 18 21:40:06 2013 +0000
----------------------------------------------------------------------
.../forgeimporters/trac/tests/test_tickets.py | 35 ++++++++++++++------
ForgeImporters/forgeimporters/trac/tickets.py | 9 +++--
2 files changed, 32 insertions(+), 12 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/50d80744/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tests/test_tickets.py b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
index 77ebdbc..c92b028 100644
--- a/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
+++ b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
@@ -155,16 +155,31 @@ class TestTracImportSupport(TestCase):
def test_link_processing(self):
import_support = TracImportSupport()
import_support.get_slug_by_id = lambda ticket, comment: '123'
- result = import_support.link_processing('''\
- test link [[2496]](http://testlink.com)
- test ticket ([#201](http://sourceforge.net/apps/trac/sourceforge/ticket/201))
- Replying to [someuser](http://sourceforge.net/apps/trac/sourceforge/ticket/204#comment:1)
- #200 unchanged''')
-
- assert "test link [\[2496\]](http://testlink.com)" in result
- assert 'test ticket ([#201](201))' in result
- assert 'Replying to [someuser](204/#123)' in result
- assert '#200 unchanged' in result, result
+ cases = {
+ 'test link [[2496]](http://testlink.com)':
+ "test link [\[2496\]](http://testlink.com)",
+
+ 'test ticket ([#201](http://site.net/apps/trac/project/ticket/201))':
+ 'test ticket ([#201](201))',
+
+ 'Replying to [someuser](http://site.net/apps/trac/project/ticket/204#comment:1)':
+ 'Replying to [someuser](204/#123)',
+
+ '**description** modified ([diff](http://site.net/apps/trac/project/ticket/205?action=diff&version=1))':
+ '**description** modified ([diff](205))',
+
+ 'Fixed in [r1000](http://site.net/apps/trac/project/changeset/1000)':
+ 'Fixed in [r1000](r1000)',
+
+ '[[Double brackets]](1) the [[whole way]](2).':
+ '[\[Double brackets\]](1) the [\[whole way\]](2).',
+
+ '#200 unchanged':
+ '#200 unchanged',
+ }
+ for input, expected in cases.items():
+ actual = import_support.link_processing(input)
+ self.assertEqual(actual, expected)
class TestTracImportSupportFunctional(TestRestApiBase, TestCase):
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/50d80744/ForgeImporters/forgeimporters/trac/tickets.py
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tickets.py b/ForgeImporters/forgeimporters/trac/tickets.py
index d86f965..4fd8fdf 100644
--- a/ForgeImporters/forgeimporters/trac/tickets.py
+++ b/ForgeImporters/forgeimporters/trac/tickets.py
@@ -189,6 +189,9 @@ class TracImportSupport(ImportSupport):
else:
return text
+ def changeset_link(self, m):
+ return '(r%s)' % m.group(1)
+
def brackets_escaping(self, m):
"""Escape double brackets."""
return '[\[%s\]]' % m.groups()[0]
@@ -203,11 +206,13 @@ class TracImportSupport(ImportSupport):
"""
comment_pattern = re.compile('\[(\S*\s*\S*)\]\(\S*/(\d+\n*\d*)#comment:(\d+)\)')
- ticket_pattern = re.compile('(?<=\])\(\S*ticket/(\d+)\)')
- brackets_pattern = re.compile('\[\[(.*)\]\]')
+ ticket_pattern = re.compile('(?<=\])\(\S*ticket/(\d+)(?:\?[^)]*)?\)')
+ changeset_pattern = re.compile(r'(?<=\])\(\S*/changeset/(\d+)(?:\?[^]]*)?\)')
+ brackets_pattern = re.compile('\[\[([^]]*)\]\]')
text = comment_pattern.sub(self.comment_link, text)
text = ticket_pattern.sub(self.ticket_link, text)
+ text = changeset_pattern.sub(self.changeset_link, text)
text = brackets_pattern.sub(self.brackets_escaping, text)
return text
[3/6] git commit: [#6836] Converted Trac ticket import to use HTML
view to fix missing line-break issue
Posted by jo...@apache.org.
[#6836] Converted Trac ticket import to use HTML view to fix missing line-break issue
Signed-off-by: Cory Johns <cj...@slashdotmedia.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/8f9a2e3b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/8f9a2e3b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/8f9a2e3b
Branch: refs/heads/cj/6837
Commit: 8f9a2e3ba54a42c10f15c2d5da3c2906601b9735
Parents: 1280f02
Author: Cory Johns <cj...@slashdotmedia.com>
Authored: Wed Nov 13 18:02:05 2013 +0000
Committer: Dave Brondsema <db...@slashdotmedia.com>
Committed: Mon Nov 18 19:57:59 2013 +0000
----------------------------------------------------------------------
Allura/allura/scripts/trac_export.py | 20 +-
.../trac/tests/data/test-list.csv | 8 +
.../trac/tests/data/test-list.html | 343 +++++++++++++++++++
.../forgeimporters/trac/tests/test_tickets.py | 25 +-
4 files changed, 387 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/8f9a2e3b/Allura/allura/scripts/trac_export.py
----------------------------------------------------------------------
diff --git a/Allura/allura/scripts/trac_export.py b/Allura/allura/scripts/trac_export.py
index bdde5b0..9377df3 100644
--- a/Allura/allura/scripts/trac_export.py
+++ b/Allura/allura/scripts/trac_export.py
@@ -146,19 +146,23 @@ class TracExport(object):
ticket_fields['class'] = 'ARTIFACT'
ticket = self.remap_fields(ticket_fields)
- # Use RSS export to get ticket description and comments
+ # Use HTML export to get ticket description and comments
import html2text
html2text.BODY_WIDTH = 0
- url = self.full_url(self.TICKET_URL % id, 'rss')
+ url = self.full_url(self.TICKET_URL % id)
self.log_url(url)
- d = feedparser.parse(urlopen(url))
- ticket['description'] = html2text.html2text(d.feed.description)
+ d = BeautifulSoup(urlopen(url))
+ desc = d.find('div', 'description').find('div', 'searchable')
+ ticket['description'] = html2text.html2text(desc.renderContents('utf8').decode('utf8')) if desc else ''
comments = []
- for comment in d['entries']:
+ for comment in d.findAll('form', action='#comment'):
c = {}
- c['submitter'] = getattr(comment, 'author', None)
- c['date'] = comment.updated_parsed
- c['comment'] = html2text.html2text(comment.summary)
+ c['submitter'] = re.sub(r'.* by ', '', comment.find('h3', 'change').text).strip()
+ c['date'] = self.trac2z_date(comment.find('a', 'timeline')['title'].replace(' in Timeline', ''))
+ changes = str(comment.find('ul', 'changes') or '')
+ body = comment.find('div', 'comment')
+ body = body.renderContents('utf8').decode('utf8') if body else ''
+ c['comment'] = html2text.html2text(changes + body)
c['class'] = 'COMMENT'
comments.append(c)
ticket['comments'] = comments
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/8f9a2e3b/ForgeImporters/forgeimporters/trac/tests/data/test-list.csv
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tests/data/test-list.csv b/ForgeImporters/forgeimporters/trac/tests/data/test-list.csv
new file mode 100644
index 0000000..516977d
--- /dev/null
+++ b/ForgeImporters/forgeimporters/trac/tests/data/test-list.csv
@@ -0,0 +1,8 @@
+id,summary,reporter,owner,description,type,status,priority,milestone,component,version,resolution,keywords,cc
+390,Closing main view does not clear list of scans,hendrikweisser,hroest,"To reproduce:
+- open an mzML file
+- scans are listed in the ""Scan view"" to the right
+- close the main view (right-click on the tab with the filename, select ""Close"")
+- main view closes, but ""Scan view"" is not cleared
+
+By itself, this is not a big problem. However, if the ""Identification view"" was used before closing the main file, switching from ""Scan view"" to ""Identification view"" in the ""Views"" window will crash TOPPView.",defect,closed,major,Good Ideas for upcoming releases,TOPPView,1.9,fixed,,timosachsenberg
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/8f9a2e3b/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tests/data/test-list.html b/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
new file mode 100644
index 0000000..9c76868
--- /dev/null
+++ b/ForgeImporters/forgeimporters/trac/tests/data/test-list.html
@@ -0,0 +1,343 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml">
+
+
+
+
+
+
+
+
+ <head>
+ <title>
+ #390 (Closing main view does not clear list of scans)
+ – OpenMS
+ </title>
+ <link rel="search" href="/apps/trac/open-ms/search" />
+ <link rel="prev" href="/apps/trac/open-ms/ticket/389" title="Ticket #389" />
+ <link rel="last" href="/apps/trac/open-ms/ticket/633" title="Ticket #633" />
+ <link rel="help" href="/apps/trac/open-ms/wiki/TracGuide" />
+ <link rel="alternate" href="/apps/trac/open-ms/ticket/390?format=csv" type="text/csv" class="csv" title="Comma-delimited Text" /><link rel="alternate" href="/apps/trac/open-ms/ticket/390?format=tab" type="text/tab-separated-values" class="tab" title="Tab-delimited Text" /><link rel="alternate" href="/apps/trac/open-ms/ticket/390?format=rss" type="application/rss+xml" class="rss" title="RSS Feed" />
+ <link rel="next" href="/apps/trac/open-ms/ticket/391" title="Ticket #391" />
+ <link rel="start" href="/apps/trac/open-ms/wiki" />
+ <link rel="stylesheet" href="/apps/trac/open-ms/chrome/common/css/trac.css" type="text/css" /><link rel="stylesheet" href="/apps/trac/open-ms/chrome/common/css/ticket.css" type="text/css" />
+ <link rel="first" href="/apps/trac/open-ms/ticket/1" title="Ticket #1" />
+ <link rel="shortcut icon" href="/apps/trac/open-ms/chrome/common/trac.ico" type="image/x-icon" />
+ <link rel="icon" href="/apps/trac/open-ms/chrome/common/trac.ico" type="image/x-icon" />
+ <link type="application/opensearchdescription+xml" rel="search" href="/apps/trac/open-ms/search/opensearch" title="Search OpenMS" />
+ <script type="text/javascript" src="/apps/trac/open-ms/chrome/common/js/jquery.js"></script><script type="text/javascript" src="/apps/trac/open-ms/chrome/common/js/trac.js"></script><script type="text/javascript" src="/apps/trac/open-ms/chrome/common/js/search.js"></script>
+ <!--[if lt IE 7]>
+ <script type="text/javascript" src="/apps/trac/open-ms/chrome/common/js/ie_pre7_hacks.js"></script>
+ <![endif]-->
+ <script type="text/javascript" src="/apps/trac/open-ms/chrome/common/js/wikitoolbar.js"></script><script type="text/javascript">
+ jQuery(document).ready(function($) {
+ $("div.description").find("h1,h2,h3,h4,h5,h6").addAnchor("Link to this section");
+ $("#changelog h3.change").addAnchor("Link to this change");
+ /* only enable control elements for the currently selected action */
+ var actions = $("#action input[name='action']");
+ function updateActionFields() {
+ actions.each(function () {
+ $(this).siblings().find("*[@id]").enable($(this).checked());
+ $(this).siblings().filter("*[@id]").enable($(this).checked());
+ });
+ }
+ actions.click(updateActionFields);
+ updateActionFields();
+ });
+ </script>
+ <link type="text/css" href="/apps/trac/open-ms/chrome/site/ha-css/default2.css" rel="stylesheet" />
+
+ <link rel="stylesheet" type="text/css" href="http://static.sourceforge.net/css/develop/hosted.php?secure=0&1383143209" media="all" />
+
+ <!-- BEGIN: AdSolution-Tag 4.2: Global-Code [PLACE IN HTML-HEAD-AREA!] -->
+ <!-- DoubleClick Random Number -->
+ <script type="text/javascript">
+ dfp_ord=Math.random()*10000000000000000;
+ dfp_tile = 1;
+ </script>
+
+ <!-- End DoubleClick Random Number -->
+ <!-- END: AdSolution-Tag 4.2: Global-Code -->
+
+</head>
+ <body>
+
+
+<script type="text/javascript">
+ var _gaq = _gaq || [];
+ _gaq.push(['_setAccount', 'UA-32013-36']);
+ _gaq.push(['_trackPageview']);
+ _gaq.push(['t2._setAccount', 'UA-36130941-3']);
+ _gaq.push(['t2._trackPageview']);
+
+ (function() {
+ var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
+ ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
+ var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
+ })();
+</script>
+
+<div id="sf_header">
+ <div class="sfh_n">
+ <ol class="sfh_n_ol">
+ <li class="sfh_n_li"><span><a href="/" id="sf_logo"><strong>SourceForge</strong></a></span></li>
+ <li class="sfh_n_li"><span><a href="/projects/open-ms" title="Summary page hosted on SourceForge.net">Summary</a></span></li>
+ <li class="sfh_n_li"><span><a href="/projects/open-ms/files" title="Files available for download">Files</a></span></li> <li class="sfh_n_li"><span><a href="/projects/open-ms/support">Support</a></span></li>
+ <li class="sfh_n_li"><span><a href="/support">Report Spam</a></span></li>
+ <li class="sfh_n_li right"><span><a href="/user/registration/" title="Get a SourceForge.net account">Create account</a></span></li>
+ <li class="sfh_n_li right"><span><a href="/account/login.php" title="Log in and gain access to additional features">Log in</a></span></li>
+ </ol>
+ </div>
+</div>
+<div id="fad1happ" class="ads">
+ <script type="text/javascript">
+ //<![CDATA[
+ document.write('<script src="http://ad.doubleclick.net/adj/ostg.sourceforge/cons_hosted_apps_p11_spons;pg=default;sz=728x90;tile='+dfp_tile+';tpc=project;tpc=open-ms;ord='+dfp_ord+'?" type="text/javascript"><\/script>');
+ dfp_tile++;
+ //]]>
+ </script>
+</div>
+
+
+ <div id="doc3" class="yui-t6">
+ <div id="bd">
+ <a name="content"></a>
+<!-- End Header -->
+ <div id="banner">
+ <div id="header">
+ <h1><a href="http://sourceforge.net/projects/open-ms/">OpenMS</a></h1>
+ </div>
+ <form id="search" action="/apps/trac/open-ms/search" method="get">
+ <div>
+ <label for="proj-search">Search:</label>
+ <input type="text" id="proj-search" name="q" size="18" value="" />
+ <input type="submit" value="Search" />
+ </div>
+ </form>
+ <div id="metanav" class="nav">
+ <ul>
+ <li class="first"><a href="/apps/trac/open-ms/prefs">Preferences</a></li><li><a href="/apps/trac/open-ms/wiki/TracGuide">Help/Guide</a></li><li class="last"><a href="/apps/trac/open-ms/about">About Trac</a></li>
+ </ul>
+ </div>
+ </div>
+ <div id="mainnav" class="nav">
+ <ul>
+ <li class="first"><a href="/apps/trac/open-ms/wiki">Wiki</a></li><li><a href="/apps/trac/open-ms/timeline">Timeline</a></li><li><a href="/apps/trac/open-ms/roadmap">Roadmap</a></li><li><a href="/apps/trac/open-ms/browser">Browse Source</a></li><li class="active"><a href="/apps/trac/open-ms/report">View Tickets</a></li><li class="last"><a href="/apps/trac/open-ms/search">Search</a></li>
+ </ul>
+ </div>
+ <div id="main">
+ <div id="ctxtnav" class="nav">
+ <h2>Context Navigation</h2>
+ <ul>
+ <li class="first "><span>← <a class="prev" href="/apps/trac/open-ms/ticket/389" title="Ticket #389">Previous Ticket</a></span></li><li class="last"><span><a class="next" href="/apps/trac/open-ms/ticket/391" title="Ticket #391">Next Ticket</a> →</span></li>
+ </ul>
+ <hr />
+ </div>
+ <div id="content" class="ticket">
+ <h1>
+ Ticket #390
+ <span class="status">(closed defect: fixed)</span>
+ </h1>
+ <!-- Do not show the ticket (pre)view when the user first comes to the "New Ticket" page.
+ Wait until they hit preview. -->
+ <!-- Ticket Box (ticket fields along with description) -->
+ <div id="ticket">
+ <div class="date">
+ <p>Opened <a class="timeline" href="/apps/trac/open-ms/timeline?from=2012-01-24T14%3A35%3A23Z%2B0000&precision=second" title="2012-01-24T14:35:23Z+0000 in Timeline">22 months</a> ago</p>
+ <p>Last modified <a class="timeline" href="/apps/trac/open-ms/timeline?from=2012-10-17T16%3A33%3A22Z%2B0000&precision=second" title="2012-10-17T16:33:22Z+0000 in Timeline">13 months</a> ago</p>
+ </div>
+ <!-- use a placeholder if it's a new ticket -->
+ <h2 class="summary searchable">Closing main view does not clear list of scans</h2>
+ <table class="properties">
+ <tr>
+ <th id="h_reporter">Reported by:</th>
+ <td headers="h_reporter" class="searchable">hendrikweisser</td>
+ <th id="h_owner">Owned by:</th>
+ <td headers="h_owner">hroest
+ </td>
+ </tr>
+ <tr>
+ <th id="h_priority">
+ Priority:
+ </th>
+ <td headers="h_priority">
+ major
+ </td>
+ <th id="h_milestone">
+ Milestone:
+ </th>
+ <td headers="h_milestone">
+ <a class="milestone" href="/apps/trac/open-ms/milestone/Good%20Ideas%20for%20upcoming%20releases">Good Ideas for upcoming releases</a>
+ </td>
+ </tr><tr>
+ <th id="h_component">
+ Component:
+ </th>
+ <td headers="h_component">
+ TOPPView
+ </td>
+ <th id="h_version">
+ Version:
+ </th>
+ <td headers="h_version">
+ 1.9
+ </td>
+ </tr><tr>
+ <th id="h_keywords">
+ Keywords:
+ </th>
+ <td headers="h_keywords" class="searchable">
+ </td>
+ <th id="h_cc">
+ Cc:
+ </th>
+ <td headers="h_cc" class="searchable">
+ timosachsenberg
+ </td>
+ </tr>
+ </table>
+ <div class="description">
+ <h3 id="comment:description">
+ Description
+ </h3>
+ <div class="searchable">
+ <p>
+To reproduce:<br />
+- open an mzML file<br />
+- scans are listed in the "Scan view" to the right<br />
+- close the main view (right-click on the tab with the filename, select "Close")<br />
+- main view closes, but "Scan view" is not cleared<br />
+</p>
+<p>
+By itself, this is not a big problem. However, if the "Identification view" was used before closing the main file, switching from "Scan view" to "Identification view" in the "Views" window will crash TOPPView.<br />
+</p>
+
+ </div>
+ </div>
+ </div>
+ <h2>Change History</h2>
+ <div id="changelog">
+ <form method="get" action="#comment" class="printableform">
+ <div class="change">
+ <h3 class="change" id="comment:1">
+ Changed <a class="timeline" href="/apps/trac/open-ms/timeline?from=2012-01-24T15%3A16%3A55Z%2B0000&precision=second" title="2012-01-24T15:16:55Z+0000 in Timeline">22 months</a> ago by cbielow
+ </h3>
+ <div class="comment searchable">
+ <p>
+duplicate of:<br />
+- <a class="closed ticket" href="/apps/trac/open-ms/ticket/316" title="defect: SpectraViewWidget is not cleaned up when last layer is deleted (closed: duplicate)">#316</a><br />
+- <a class="closed ticket" href="/apps/trac/open-ms/ticket/283" title="defect: TOPPView: Scan view widget does not clear after closing last file (closed: duplicate)">#283</a><br />
+</p>
+<p>
+Closing the other two as this is the most comprehensive description!<br />
+</p>
+
+ </div>
+ </div>
+ </form><form method="get" action="#comment" class="printableform">
+ <div class="change">
+ <h3 class="change" id="comment:2">
+ Changed <a class="timeline" href="/apps/trac/open-ms/timeline?from=2012-09-18T08%3A30%3A38Z%2B0000&precision=second" title="2012-09-18T08:30:38Z+0000 in Timeline">14 months</a> ago by hroest
+ </h3>
+ <ul class="changes">
+ <li>
+ <strong>milestone</strong>
+ changed from <em>Bug-Reports</em> to <em>Release 1.10</em>
+ </li>
+ </ul>
+ <div class="comment searchable">
+ <p>
+maybe related to <a class="closed ticket" href="/apps/trac/open-ms/ticket/391" title="defect: TOPPView crashes when opening a file in Identification view (closed: fixed)">#391</a> which is also a problem with the identification view?<br />
+</p>
+
+ </div>
+ </div>
+ </form><form method="get" action="#comment" class="printableform">
+ <div class="change">
+ <h3 class="change" id="comment:3">
+ Changed <a class="timeline" href="/apps/trac/open-ms/timeline?from=2012-09-18T11%3A46%3A21Z%2B0000&precision=second" title="2012-09-18T11:46:21Z+0000 in Timeline">14 months</a> ago by aiche
+ </h3>
+ <ul class="changes">
+ <li>
+ <strong>owner</strong>
+ changed from <em>all</em> to <em>hroest</em>
+ </li><li>
+ <strong>status</strong>
+ changed from <em>new</em> to <em>assigned</em>
+ </li><li>
+ <strong>milestone</strong>
+ changed from <em>Release 1.10</em> to <em>Good Ideas for upcoming releases</em>
+ </li>
+ </ul>
+ <div class="comment searchable">
+
+ </div>
+ </div>
+ </form><form method="get" action="#comment" class="printableform">
+ <div class="change">
+ <h3 class="change" id="comment:4">
+ Changed <a class="timeline" href="/apps/trac/open-ms/timeline?from=2012-10-17T16%3A33%3A22Z%2B0000&precision=second" title="2012-10-17T16:33:22Z+0000 in Timeline">13 months</a> ago by timosachsenberg
+ </h3>
+ <ul class="changes">
+ <li>
+ <strong>status</strong>
+ changed from <em>assigned</em> to <em>closed</em>
+ </li><li>
+ <strong>resolution</strong>
+ set to <em>fixed</em>
+ </li>
+ </ul>
+ <div class="comment searchable">
+ <p>
+fixed in rev. 10330<br />
+</p>
+
+ </div>
+ </div>
+ </form>
+ </div>
+ <div id="help">
+ <strong>Note:</strong> See
+ <a href="/apps/trac/open-ms/wiki/TracTickets">TracTickets</a> for help on using
+ tickets.
+ </div>
+ </div>
+ <div id="altlinks">
+ <h3>Download in other formats:</h3>
+ <ul>
+ <li class="first">
+ <a rel="nofollow" href="/apps/trac/open-ms/ticket/390?format=csv" class="csv">Comma-delimited Text</a>
+ </li><li>
+ <a rel="nofollow" href="/apps/trac/open-ms/ticket/390?format=tab" class="tab">Tab-delimited Text</a>
+ </li><li class="last">
+ <a rel="nofollow" href="/apps/trac/open-ms/ticket/390?format=rss" class="rss">RSS Feed</a>
+ </li>
+ </ul>
+ </div>
+ </div>
+ <div id="footer" lang="en" xml:lang="en"><hr />
+ <a id="tracpowered" href="http://trac.edgewall.org/"><img src="/apps/trac/open-ms/chrome/common/trac_logo_mini.png" height="30" width="107" alt="Trac Powered" /></a>
+ <p class="left">
+ Powered by <a href="/apps/trac/open-ms/about"><strong>Trac 0.11.2.1</strong></a><br />
+ By <a href="http://www.edgewall.org/">Edgewall Software</a>.
+ </p>
+ <p class="right"></p>
+ </div>
+<!-- Footer -->
+ </div>
+ </div>
+<!-- End Footer -->
+
+
+<p class="copyright">© 2013 SourceForge. All Rights Reserved.
+SourceForge is a <a href="http://www.diceholdingsinc.com/phoenix.zhtml?c=211152&p=irol-landing">Dice Holdings, Inc.</a> company
+<a href="http://slashdotmedia.com/terms-of-use">Terms of Use</a> -
+<a href="http://slashdotmedia.com/privacy-statement/">Privacy Policy</a> -
+<a href="http://slashdotmedia.com/opt-out-choices">Cookies/Opt Out</a>
+</p>
+
+ <script type="text/javascript">
+ pageTracker._trackPageview();
+ </script>
+</body>
+</html>
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/8f9a2e3b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
----------------------------------------------------------------------
diff --git a/ForgeImporters/forgeimporters/trac/tests/test_tickets.py b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
index f55ac67..77ebdbc 100644
--- a/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
+++ b/ForgeImporters/forgeimporters/trac/tests/test_tickets.py
@@ -17,9 +17,10 @@
import json
import os
+from datetime import datetime
from unittest import TestCase
-from mock import Mock, patch
+from mock import Mock, patch, MagicMock
from ming.orm import ThreadLocalORMSession
from pylons import tmpl_context as c
@@ -203,3 +204,25 @@ class TestTracImportSupportFunctional(TestRestApiBase, TestCase):
import_support = TracImportSupport()
self.assertEqual(import_support.get_slug_by_id('204', '1'), comments[0].slug)
self.assertEqual(import_support.get_slug_by_id('204', '2'), comments[1].slug)
+
+ @with_tracker
+ def test_list(self):
+ from allura.scripts.trac_export import TracExport, DateJSONEncoder
+ csv_fp = open(os.path.dirname(__file__) + '/data/test-list.csv')
+ html_fp = open(os.path.dirname(__file__) + '/data/test-list.html')
+ with patch.object(TracExport, 'next_ticket_ids', return_value=[(390, {})]):
+ te = TracExport('url', do_attachments=False)
+ te.exhausted = True
+ te.csvopen = lambda s: csv_fp
+ with patch('allura.scripts.trac_export.urlopen', return_value=html_fp):
+ json_data = {
+ 'class': 'PROJECT',
+ 'trackers': {'default': {'artifacts': list(te)}},
+ }
+ TracImportSupport().perform_import(
+ json.dumps(json_data, cls=DateJSONEncoder),
+ '{"user_map": {}}')
+ ticket = TM.Ticket.query.get(app_config_id=c.app.config._id,
+ ticket_num=390)
+ self.assertIn('To reproduce: \n\\- open an mzML file', ticket.description)
+ self.assertIn('duplicate of: \n\\- [#316](316)', ticket.discussion_thread.find_posts()[0].text)
[4/6] git commit: [#6836] Add new test data files to rat-exclude
Posted by jo...@apache.org.
[#6836] Add new test data files to rat-exclude
Signed-off-by: Cory Johns <cj...@slashdotmedia.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/e3b8453b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/e3b8453b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/e3b8453b
Branch: refs/heads/cj/6837
Commit: e3b8453bfa16d7129afeddf858ac0d5c16208eea
Parents: 8f9a2e3
Author: Cory Johns <cj...@slashdotmedia.com>
Authored: Mon Nov 18 20:28:25 2013 +0000
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Mon Nov 18 20:28:25 2013 +0000
----------------------------------------------------------------------
rat-excludes.txt | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/e3b8453b/rat-excludes.txt
----------------------------------------------------------------------
diff --git a/rat-excludes.txt b/rat-excludes.txt
index 968c66f..cc89a91 100644
--- a/rat-excludes.txt
+++ b/rat-excludes.txt
@@ -38,3 +38,5 @@ ForgeGit/forgegit/data/post-receive_tmpl
ForgeSVN/forgesvn/tests/data/
ForgeImporters/forgeimporters/tests/data/google/empty-issue.html
ForgeImporters/forgeimporters/tests/data/google/test-issue.html
+ForgeImporters/forgeimporters/trac/tests/data/test-list.csv
+ForgeImporters/forgeimporters/trac/tests/data/test-list.html