Merge pull request #94 from pubs/fix/93
Fix #93: actually save the modifications on `edit -m`.
This commit is contained in:
commit
f6035cac80
@ -51,6 +51,7 @@ def command(conf, args):
|
|||||||
if meta:
|
if meta:
|
||||||
new_paper = Paper(paper.citekey, paper.bibdata,
|
new_paper = Paper(paper.citekey, paper.bibdata,
|
||||||
metadata=content)
|
metadata=content)
|
||||||
|
rp.push_paper(new_paper, overwrite=True, event=False)
|
||||||
else:
|
else:
|
||||||
new_paper = Paper.from_bibentry(content,
|
new_paper = Paper.from_bibentry(content,
|
||||||
metadata=paper.metadata)
|
metadata=paper.metadata)
|
||||||
@ -62,7 +63,7 @@ def command(conf, args):
|
|||||||
choice = options[ui.input_choice(
|
choice = options[ui.input_choice(
|
||||||
options, ['o', 'e', 'a'],
|
options, ['o', 'e', 'a'],
|
||||||
question='A paper already exists with this citekey.'
|
question='A paper already exists with this citekey.'
|
||||||
)]
|
)]
|
||||||
|
|
||||||
if choice == 'abort':
|
if choice == 'abort':
|
||||||
break
|
break
|
||||||
|
@ -579,6 +579,20 @@ class TestUsecase(DataCommandTestCase):
|
|||||||
]
|
]
|
||||||
self.execute_cmds(cmds)
|
self.execute_cmds(cmds)
|
||||||
|
|
||||||
|
def test_edit_meta(self):
|
||||||
|
meta = str_fixtures.turing_meta
|
||||||
|
|
||||||
|
line = '[Page99] Page, Lawrence et al. "The PageRank Citation Ranking: Bringing Order to the Web." (1999) \n'
|
||||||
|
line1 = re.sub('\n', '| AI,computer\n', line)
|
||||||
|
|
||||||
|
cmds = ['pubs init',
|
||||||
|
'pubs add data/pagerank.bib',
|
||||||
|
('pubs list', [], line),
|
||||||
|
('pubs edit -m Page99', [meta]),
|
||||||
|
('pubs list', [], line1),
|
||||||
|
]
|
||||||
|
self.execute_cmds(cmds)
|
||||||
|
|
||||||
def test_export(self):
|
def test_export(self):
|
||||||
cmds = ['pubs init',
|
cmds = ['pubs init',
|
||||||
('pubs add', [str_fixtures.bibtex_external0]),
|
('pubs add', [str_fixtures.bibtex_external0]),
|
||||||
|
Loading…
x
Reference in New Issue
Block a user