On Wed, Nov 5, 2014 at 2:21 AM, Christian Egli <christian.e...@sbs.ch> wrote: > John Hendy <jw.he...@gmail.com> writes: > >> I ran into an odd issue with a taskjuggler task tree I'm working on >> when trying to apply a gapduration attribute to a task dependency. >> Typically, I can just pass any valid taskjuggler attribute through >> using properties, but my export was producing an error when trying to >> use: >> >> :depends: task {gapduration 1h} >> >> Oddly, the output is as follows (task section): >> >> #+begin_src test.tjp >> >> task test "test" { >> task task1 "task1" { >> milestone >> start 2014-11-05-08:00 >> } >> task task2 "task2" { >> depends !task1 t :taskjuggl >> duration 1h >> } >> #+end_src >> >> For some reason it's grabbing the taskjuggler tag! I tried with the >> example in ox-taskjuggler.el with the same result. >> >> Is this a bug or am I mis-interpreting how this would work? > > No it is indeed a bug. At first I thought this was simply no longer > working in the new exporter but I was wrong. Nicolas of course > implemented it and way more. But there is a subtle bug. I actually found > it because I wondered about the funny "t :taskjuggl" in the output. > Here's the fix. > > diff --git a/contrib/lisp/ox-taskjuggler.el b/contrib/lisp/ox-taskjuggler.el > index 807d702..9e977f6 100644 > --- a/contrib/lisp/ox-taskjuggler.el > +++ b/contrib/lisp/ox-taskjuggler.el > @@ -600,7 +600,7 @@ doesn't include leading \"depends\"." > (let ((id (org-element-property :TASK_ID dep))) > (and id > (string-match (concat id " +\\({.*?}\\)") dep-str) > - (org-match-string-no-properties 1)))) > + (org-match-string-no-properties 1 dep-str)))) > path) > ;; Compute number of exclamation marks by looking for the > ;; common ancestor between TASK and DEP. >
For some reason, I couldn't apply the patch... since it was just one line, I manually changed it myself and can report that this appears to work now. I re-added the {gapduration x} lines to my :depends: properties and they're being properly applied now. Thanks a ton for the quick turnaround! If/when you apply this, could you post back and I'll checkout a fresh version of ox-taskjuggler to take the official commit? Best regards, John > Nicolas, I can push this myself but I haven't pushed to the repo in > years. What is the current way? Do I just push to master? > > Thanks > Christian > -- > Christian Egli > Swiss Library for the Blind, Visually Impaired and Print Disabled > Grubenstrasse 12, CH-8045 Zürich, Switzerland > >