Last Comment Bug 730704 - kumascript: Changes to Kuma wiki document view to use Kumascript service
: kumascript: Changes to Kuma wiki document view to use Kumascript service
u=developer c=wiki p=3 2.4.5
Product: Mozilla Developer Network
Classification: Other
Component: Editing (show other bugs)
: unspecified
: x86 Mac OS X
: -- normal (vote)
: 2.4
Assigned To: Les Orchard [:lorchard]
Depends on: 730705 730706
Blocks: 659364 730714
  Show dependency treegraph
Reported: 2012-02-26 11:43 PST by Les Orchard [:lorchard]
Modified: 2012-09-19 00:56 PDT (History)
1 user (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Description User image Les Orchard [:lorchard] 2012-02-26 11:43:56 PST
Kuma, on the Django side, needs modifications to use the node.js kumascript service on document viewing and previewing.

This is what I *think* needs doing in wiki.views.document():

* The code that handles redirects and translation stays the same.
* Leave the code path handling the ?raw option alone.
* If no ?raw parameter: Perform an HTTP GET to the kumascript service to get document HTML, rather than access doc.html directly

The kumascript service should, in turn, request the doc source with ?raw and respond with macros evaluated. On the Django side, that content should still be processed for sections and editing links, etc
Comment 1 User image Les Orchard [:lorchard] 2012-02-28 10:52:59 PST
Going ahead and adding this to 2.4.5 with 3 pts. I'm already underway with this and some ancillary tasks it brought up:
Comment 2 User image [github robot] 2012-03-05 12:57:55 PST
Commits pushed to mdn at
bug 730704: Changes to kuma to use kumascript service

* Document view performs a request to the kumascript service to evaluate
  macros before rendering response

* Tweak to CKEditor setup, skips setting it up on Template:* pages in
  advance of more work for bug 731651

* Tests for kumascript usage in document view

* Constance config setting to enforce kumascript request timeout (or to
  disable entirely, which is the default)

* Tweaks to document view and model to support HTTP ETag and
  Last-Modified headers, in advance of caching support for
  bug 730714 and bug 730715

* Add kumascript service as a submodule

* Add node.js and kumascript config to vagrant/puppet

* Added tmux to vagrant install, because it's very handy

* Updates to .gitignore

* Bugfix to 404 on js/libs/jqueryui-min.js
bug 730704: advise using forked runserver in MOTD, kumascript update
Merge pull request #126 from lmorchard/bug-730704-kumascript-integration

Bug 730704 kumascript integration
Comment 3 User image Luke Crouch [:groovecoder] 2012-03-06 13:22:27 PST
Depends on bug 730705 for QA. Let's leave this resolved and get it onto stage9 during 2.5 for QA.

Note You need to log in before you can comment on or make changes to this bug.