27 Jun
2017
27 Jun
'17
10:29 a.m.
vmassol wrote
Some questions/remarks:
* I still need to review your code in the coming few days
* Does it work? Am I able to get a RedPen validation when I save a page?
(I need to test your extension now)
* In the ConfigurableClass, you should use a static list of Strings that
lets the user enter the exclusions so that they can enter regex. Something
similar to
http://extensions.xwiki.org/xwiki/bin/view/Extension/Multiselect%20custom%2…
I think this would be more powerful. IMO you should also have includes and
not just excludes to cover all use cases.
* I see that you have implemented a UI to allow entering some words to be
corrected. Does it mean that the RedPen execution not only validates
content but is also able to modify it?
Thanks for the update!
-Vincent
Hi! Thanks in advance for the review :)
I am still working through issues in my code one at a time, and I don't
think it would be working today. I will keep you updated on it, and if need
be I can commit an earlier version, without much of the UI capabilities,
which validates plain text on a page on save and appends the results at the
bottom of the page.
After getting my code to work I'll edit the configurable class. I can see
that includes and excludes would cover all use cases. I will work on editing
the class such that the user can choose to include or exclude.
RedPen can take in a dictionary of words or expressions(in JSON format), but
it doesn't alter the entries directly. Rather, it would output a message
along with the other validation messages, with this one suggesting the word
or expression replacement.
However, I think it is possible to implement a replacement function. Since
the validation message provides the location of the error, we can extract
the erroreous word or expression out and replace it with the corrected one.
What do you think?
--
View this message in context:
http://xwiki.475771.n2.nabble.com/GSOC-Update-3-RedPen-Integration-tp760425…
Sent from the XWiki- Dev mailing list archive at Nabble.com.