search

collective / stxnext.staticdeployment

10 stars 4 forks source link

use xml parsing instead of regular expressions and text replacement #7

Open vangheem opened 11 years ago

vangheem commented 11 years ago

This is more something that should be done long term in the product that I might contribute to doing eventually.

Right now you can run into problems where the transformation of one url can also change others that partially match that url. If we moved all transforms to use lxml instead of regular expressions, it'd fix this issue. I had to do an ugly hack for now though...

Also, we should probably use lxml instead of beautiful soup since it's pretty standard.

radekj commented 11 years ago

+1

ctxlken commented 11 years ago

Not to mention, lxml should be quite a bit faster, I would think (C vs. Python.)