2010/6/9 Hannes Magnusson <hannes.magnus...@gmail.com>

> On Tue, Jun 8, 2010 at 19:38, Yannick Torrès <yannick.tor...@gmail.com>
> wrote:
> > Hi all,
> >
> > I'm about to commit a new functionnality for the editor : Checking the
> Xml
> > with Xmllint.
> >
> > In a first time, I put this functionnality as "optionnal" as this allow
> the
> > user to decide to check his file before save/commit it.
> >
> > If the result in production is good, I hope we can use it as default when
> > the user save his file. As it, no more documentation break will be done
> via
> > the PhDOE ;)
> >
> > Anyways, actually, the xmllint binary is not available into pb11. Can I
> > install it before commit this new functionnality ?
>
> Sure. You do have access to the box, don't you?
>

Yes I have. Just install it via aptitude.



>
> I'm curious how you are gonna implement this.. You cannot validate
> individual files as they simply aren't valid standalone xml files, and
> you shouldn't be validating manual.xml as that will take ages for
> every save...
>

I just commit this :
http://svn.php.net/viewvc?view=revision&revision=300323

About the processus, you're right, it's not a valid xml file as I don't
validate the entities.
As it, I just make somethings simple as I pass the xml to xmllint and filter
all entities check warning to display only others errors.
When a use make a mistake, it's usually on a tag (extra/typo/remove tag).
With this, we can hope to avoid a lot of mistakes.

You can view the result as I update edit.php.net too.
Just open a file in any module, make a mistake on a tag and click on "Check
XML with Xmllint" button, left on "enable spellchecking" button in the
toolbar.



>
> -Hannes
>

Yannick

Reply via email to