po(scan): make more clear what's being done

Mostly comments, and also a small code change: simplification and implementation
details hiding.

Signed-off-by: intrigeri <intrigeri@boum.org>

po: reorder nearly all of the module code

It has grown up incrementally and new helper functions were added right in the
middle of the hooks, most often near the place they were used, which is
practical when doing initial development, but quite ugly afterwards, when helper
functions are useful to separate logic and implementation details.

Today's refactoring commits have brought the code to a much more maintainable
state, IMHO.

Signed-off-by: intrigeri <intrigeri@boum.org>

po/doc: first fuzzy-testing results for po4a and msgmerge

Chapter #1, in which we learn po4a could help to DoS ikiwiki + po,
whereas msgmerge seems reluctant to cooperate.

Signed-off-by: intrigeri <intrigeri@boum.org>

po/doc: more security research results

Apart of the fuzzying part, I'm done with what I can do without help.

The "Running po4a on untrusted content" section needs at least a quick glance
from an experimented Perl programmer.

Signed-off-by: intrigeri <intrigeri@boum.org>

po(match_lang, match_currentlang): use the lang() function

... now that it exists, instead of duplicating it.

Signed-off-by: intrigeri <intrigeri@boum.org>

po: rewrote otherlanguagesloop()

It is now more elegant IMHO, and the output is now sorted according to the
language name (instead of code).

Signed-off-by: intrigeri <intrigeri@boum.org>

po: factorize extraction of a slave page's masterpage and lang

The very same code was repeated at dozens of places.

NB: the real work is now done is _istranslation(), which is memoized,
so the additional function calls overhead should be compensated.

Signed-off-by: intrigeri <intrigeri@boum.org>

t/po.t: removed debug lines

Signed-off-by: intrigeri <intrigeri@boum.org>

po(otherlanguagesloop): start factorizing

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge commit 'upstream/master' into prv/po

po: finish abstracting %filtered by putting it into a blackbox

i.e. it is invisible for everyone but its accessors/mutators

Signed-off-by: intrigeri <intrigeri@boum.org>

po: actually use pofiles() function, to enhance code clarity

Signed-off-by: intrigeri <intrigeri@boum.org>

po: abstract "doing something for the same page in every other language"

Signed-off-by: intrigeri <intrigeri@boum.org>

po: abstract implementation details for %translations building and resetting

Signed-off-by: intrigeri <intrigeri@boum.org>

po: abstract %filtered implementation details

Signed-off-by: intrigeri <intrigeri@boum.org>

po: clarify error messages

Signed-off-by: intrigeri <intrigeri@boum.org>

po: clarify code with some comments

Signed-off-by: intrigeri <intrigeri@boum.org>

po: prevent a slave page to depend on itself

It has never caused harm yet, but it might in the future.

Signed-off-by: intrigeri <intrigeri@boum.org>

vim syntax file linked

Fix the link() pagespec to match links that are internally recorded as absolute.

This fixes a problem exposed by the recent change to tags
(a2839de9362187b67b0e3a564461e272e64fd9b4). That recorded tag links as
absolute by including a leading slash in the link. The same could also be
done with an absolute wikilink.

In either case, link() would not match such links, unless the leading slash
was included in the link to match. But that's not right, because pagespecs
match absolute by default. So strip the leading slash.

Note that to keep any existing `link(/foo)` pagespecs working after this
change, the leading slash is removed from there, too.

po/todo(security): many research results

... and some questions to Joey (hint: look for your name)

Signed-off-by: intrigeri <intrigeri@boum.org>

po(refreshpot): define the input charset before read()'ing

... to prevent the use of Encode::Guess::guess_encoding() in
Locale::Po4a::Transtractor (just a minor security measure, dependent on po4a
internals, but we have no reason to think Encode::Guess is not safe).

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info

fix link to Jason Blevins user page

Merge branch 'prv/po' into pub/po

question about the format directive and relation with htmlize hooks

po/todo++: support other file formats than markdown

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge branch 'prv/po' into pub/po

po/todo: mostly security research

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge branch 'prv/po' into pub/po

po: started research on gettext/po4a security

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge branch 'prv/po' into pub/po

fixed link

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge branch 'prv/po' into pub/po

added symlink to pagespec/po in basewiki to fix the build

else basewiki_brokenlinks.t fails.

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge commit 'upstream/master' into pub/po

Merge commit 'upstream/master' into prv/po

po: finished backlinks implementation

Signed-off-by: intrigeri <intrigeri@boum.org>

po: oops, removed debug output from testsuit

Signed-off-by: intrigeri <intrigeri@boum.org>

po: implemented linking/backlinks specification for po_link_to=negotiated

Signed-off-by: intrigeri <intrigeri@boum.org>

po: implemented linking specification in testcase

Signed-off-by: intrigeri <intrigeri@boum.org>

po: added pofiles() function, will use this soon

Signed-off-by: intrigeri <intrigeri@boum.org>

document wikistatedir, though it's still internal

po: write detailed specification of wished backlinks behaviour

Signed-off-by: intrigeri <intrigeri@boum.org>

po: add comments

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge commit 'upstream/po' into prv/po

oops

txt: Do not encode quotes when filtering the txt, as that broke later parsing of any directives on the page.

response

meta: Plugin is now enabled by default since the basewiki uses it.

Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info

aggregate: Try to query XML::Feed for the base url when derelevatising links. Since this needs the just released XML::Feed 0.3, as well  as a not yet released XML::RSS, it will fall back to the old method if no xml:base info is available.

response

response

note copyright of po.pm

po: fix link to homepage in special case

... by wrapping IkiWiki::urlto in order to workaround hard-coded
/index.$config{htmlext}, which is wrong when usedirs=0 and po_link_to=current
and translatable homepage

Signed-off-by: intrigeri <intrigeri@boum.org>

po: misc responses to Joey

Signed-off-by: intrigeri <intrigeri@boum.org>

po: redesign (once more) automatic POT/PO files update

Now use the change hook to update these files, check them into VCS, and trigger
IkiWiki::refresh as needed. The needsbuild hook's help was required to prevent
infinite looping.

This more rigorous way of doing this fixes recentchanges (that was previously
not updated in some cases), and probably is a better long-term solution than the
two previously tested ones.

Signed-off-by: intrigeri <intrigeri@boum.org>

New bug report

po: doc/todo update

Signed-off-by: intrigeri <intrigeri@boum.org>

po: use prep_writefile before creating any files in refreshpot and refreshpo

Signed-off-by: intrigeri <intrigeri@boum.org>

po: reported parentlinks misbehaviour in special case

Signed-off-by: intrigeri <intrigeri@boum.org>

po: syntax warning fix

Signed-off-by: intrigeri <intrigeri@boum.org>

po: fixed whitespace

Signed-off-by: intrigeri <intrigeri@boum.org>

po: po_link_to is safe, and fallbacks to a working value if needed

warnings are displayed if it is set to an invalid or incompatible value
(e.g. po_link_to=negotiated and disabled usedirs)

Signed-off-by: intrigeri <intrigeri@boum.org>

po: mark po_translatable_pages as a safe configuration setting

Signed-off-by: intrigeri <intrigeri@boum.org>

po: po_master_language and po_slave_languages are safe config settings

... and check their validity in checkconfig

Signed-off-by: intrigeri <intrigeri@boum.org>

po: added license and copyright

Signed-off-by: intrigeri <intrigeri@boum.org>

po: response

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge commit 'upstream/master' into prv/po

Merge commit 'upstream/po' into prv/po

Conflicts:

IkiWiki/Plugin/po.pm
doc/plugins/po.mdwn

Signed-off-by: intrigeri <intrigeri@boum.org>

first pass over code

Only change of note is quoting some strings in a regexp, just in case
(also avoids the . matching any character!)

Mostly whitespace changes of no consequence.

backup refs to injected subs later

In case two plugins both inject replacements for these subs,
backup the ref just before injecting to make sure the most
recent version is seen.

minor layout changes

need license

response

fixed one security problem, two more need review

avoid hitting the shell with system()

response

comments and questions

minor wording improvements/changes

another reason to require rebuilds (and not be marked safe)

the plugin changes html filenames, so requiring a manual transition to
enable

add a link to pages for enhanced pagespecs

suggest po4a for po plugin

Merge commit 'intrigeri/po'

response

add a more useful example of a relative date ;-)

po plugin: more detailed backlinks todo item

Signed-off-by: intrigeri <intrigeri@boum.org>

use error for two messages

po plugin(filter): redesign temp file management

... in a way compatible with various File::Temp versions.
The result is far from being perfect (see comments in the code for details),
but it does work.

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge branch 'prv/po' into pub/po

Merge commit 'upstream/master' into prv/po

po plugin: redesign refresh mechanism after master page update

manually triggering IkiWiki::refresh() was at least dubious, and more or less
buggy (it randomly broke the whole backlinks feature); thinking a bit more to
add the necessary bits to @needsbuild seems like a better way. don't play with
ikiwiki's internals if not absolutely needed.

Signed-off-by: intrigeri <intrigeri@boum.org>

po plugin: remove broken parentlinks for home page's translations

Signed-off-by: intrigeri <intrigeri@boum.org>

po plugin: updated doc/todo

Signed-off-by: intrigeri <intrigeri@boum.org>

po plugin: aesthetics

Signed-off-by: intrigeri <intrigeri@boum.org>

maintain list