Closed (fixed)
Project:
AsciiDoc filter
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Jul 2017 at 13:16 UTC
Updated:
27 Jul 2017 at 13:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
hgoto commentedHere is a first trial patch. I'd like this to be reviewed.
Comment #3
marvil07 commentedThanks for the patch, I verified it still works with asciidoc binary.
The patch looks good, except for two minor things:
The
admin/reports/statuspage shows asciidoc version parsing output and assuming it is likeasciidoc 8.6.9, you may want to change thesubstr(array_shift($output), 9);line to report the version correctly.Typo.
I really have never used asciidoctor, but it looks like it supports the same arguments, so it should be OK to add it here.
Possible Future changes
One thing that would be great would be a way to choose among the two if the two are installed.
With the patch it chooses the first it finds prioritizing asciidoc binary, so behaviour is maintained, that's good.
I think the right place to do that is on the filter configuration, currently is it empty.
@hgoto Do you want to open a follow-up issue about that?
Comment #4
marvil07 commentedComment #5
hgoto commented@marvil07 thank you for your response!
I fixed the points you noted. And I added a small change to the requirements message so that users would know asciidoctor can be used.
As you guess, asciidoctor supports the same argument as asciidoc does as far as I checked.
The possible future changes sound good. I'll open a follow-up issue and give it a try after this issue is closed.
(I attached 2 capture images of the status page as well a patch and an interdiff file.)
Comment #7
marvil07 commentedThanks for the changes, this is now part of 8.x-1.x.
I did one minor change after adding it. See the following change notice: Removed REQUIREMENT_INFO/REQUIREMENT_OK from most hook_requirements() items for details.
Comment #9
hgoto commented@marvil07 thanks! I didn't notice the points you modified.
I'll open a follow-up issue soon...