| Commit message (Expand) | Author | Age |
|---|
| * | typo | Joey Hess | 2008-12-26 |
| * | Merge branch 'master' into next | Joey Hess | 2008-12-26 |
| |\ |
|
| | * | inline: Run format hook first•••inline has a format hook that is an optimisation hack. Until this hook
runs, the inlined content is not present on the page. This can prevent
other format hooks, that process that content, from acting on inlined
content. In bug ##509710, we discovered this happened commonly for the
embed plugin, but it could in theory happen for many other plugins (color,
cutpaste, etc) that use format to fill in special html after sanitization.
The ordering was essentially random (hash key order). That's kinda a good
thing, because hooks should be independent of other hooks and able to run
in any order. But for things like inline, that just doesn't work.
To fix the immediate problem, let's make hooks able to be registered as
running "first". There was already the ability to make them run "last".
Now, this simple first/middle/last ordering is obviously not going to work
if a lot of things need to run first, or last, since then we'll be back to
being unable to specify ordering inside those sets. But before worrying about
that too much, and considering dependency ordering, etc, observe how few
plugins use last ordering: Exactly one needs it. And, so far, exactly one
needs first ordering. So for now, KISS.
Another implementation note: I could have sorted the plugins with
first/last/middle as the primary key, and plugin name secondary, to get a
guaranteed stable order. Instead, I chose to preserve hash order. Two
opposing things pulled me toward that decision:
1. Since has order is randomish, it will ensure that no accidental
ordering assumptions are made.
2. Assume for a minute that ordering matters a lot more than expected.
Drastically changing the order a particular configuration uses could
result in a lot of subtle bugs cropping up. (I hope this assumption is
false, partly due to #1, but can't rule it out.)
| Joey Hess | 2008-12-26 |
| * | | Merge branch 'master' into next | Joey Hess | 2008-12-26 |
| |\| |
|
| | * | comments: Deal with users entering unqualified or partial urls.•••People seem to be able to expect to enter www.foo.com and get away with it.
The resulting my.wiki/www.foo.com link was not ideal.
To fix it, use URI::Heuristic to expand such things into a real url. It
even looks up hostnames in the DNS if necessary.
| Joey Hess | 2008-12-26 |
| * | | Merge branch 'master' into next•••Conflicts:
IkiWiki/Plugin/googlecalendar.pm
| Joey Hess | 2008-12-25 |
| |\| |
|
| | * | googlecalendar: Add runtime deprecation warning. | Joey Hess | 2008-12-25 |
| | * | camelcase: Add camelcase_ignore setting. | Joey Hess | 2008-12-23 |
| * | | remove deprecated googlecalendar plugin | Joey Hess | 2008-12-24 |
| * | | remove deprecated admin prefs•••A new ikiwiki-transition moveprefs subcommand can pull the old data out of
the userdb and inject it into the setup file.
Note that it leaves the old values behind in the userdb too. I did this
because I didn't want to lose data if it fails writing the setup file for
some reason, and the old data in the userdb will only use a small amount of
space. Running the command multiple times will mostly not change anything.
| Joey Hess | 2008-12-24 |
| * | | finalise version 3.00 of the plugin api | Joey Hess | 2008-12-23 |
| * | | enable aggregate_internal by default | Joey Hess | 2008-12-23 |
| |/ |
|
| * | recentchanges_link CGI: If the linked page is internal, use its permalink•••This fixes a bug, that comments appear in recentchanges as broken links.
| Simon McVittie | 2008-12-22 |
| * | Merge commit 'smcv/openid' | Joey Hess | 2008-12-21 |
| |\ |
|
| | * | openid: in &openiduser, allow subdirectory-style providers to end with '/'•••This improves the display of OpenIDs like 'http://id.mayfirst.org/jamie/'
(taking an example from the IkiWiki commit log).
| Simon McVittie | 2008-12-21 |
| | * | openid: in &openiduser, let domain-style providers have arbitrarily many subd...•••This leads to better display for OpenIDs like smcv.pseudorandom.co.uk
and thm.id.fedoraproject.org (to take a couple of examples from the
IkiWiki commit history).
| Simon McVittie | 2008-12-21 |
| * | | comments: substitute commentsurl and atomcommentsurl for use in feeds | Simon McVittie | 2008-12-21 |
| * | | comments: run pagetemplate hooks•••This fixes the bug that comments are always said to be from an anonymous
user at an unknown IP address.
| Simon McVittie | 2008-12-21 |
| * | | comments: linkify and preprocess preview with correct 'page' param | Simon McVittie | 2008-12-21 |
| |/ |
|
| * | meta: Process meta date during scan pass so that the date will always affect ... | Joey Hess | 2008-12-21 |
| * | avoid storing transient state in pagestate•••None of the comment state needs to be stored through the a later run of
ikiwiki, so move it all from pagestate to a more transient storage.
This is assuming that we'll never want to add pagespecs to search against
the comment state. Pagespecs like author() are why the meta plugin does
store its meta data in pagestate -- the data can be needed later to match
against.
| Joey Hess | 2008-12-20 |
| * | comments: Rename COMMENTURL to ADDCOMMENTURL to avoid confusion with COMMENTA...•••Also refactor page.tmpl to use if/else rather than unless/if.
| Simon McVittie | 2008-12-20 |
| * | comments: pass COMMENTOPENID to templates | Simon McVittie | 2008-12-20 |
| * | comments: remove linkuser(), it's been integrated into preprocess() now | Simon McVittie | 2008-12-20 |
| * | _comment directive: if the user looks like an OpenID, store that | Simon McVittie | 2008-12-20 |
| * | avoid uninitialized value•••This can happen when a new field,
such as the new lasttry, is added.
| Joey Hess | 2008-12-19 |
| * | fix comment permalink to always point to comment parent page | Joey Hess | 2008-12-19 |
| * | remove cruft•••wtf does it do? absolutely nothing
| Joey Hess | 2008-12-19 |
| * | rename comments_form to editcomment | Joey Hess | 2008-12-19 |
| * | rename comments_display to comment | Joey Hess | 2008-12-19 |
| * | replace discussion links on pages with comments link•••The thinking here is that having both a Discussion page and comments for
the same page is redundant, and certianly not what you want if you enable
comments for a page. At first I considered making configurable via pagespec
what pages got discussion links. But that would mean testing a new pagespec
for every page, and a redundant config setting to keep in sync. So intead,
take a lead from my previous change to make inlined pages have a comments
link, and change the discussion link at the top of regular pages to link to
their comments.
(Implementation is a bit optimised to avoid redundant pagespec checking.)
| Joey Hess | 2008-12-19 |
| * | jump to comment after posting•••Jumping to the just posted comment was the imputus, but I killed a number
of birds here.
Added a INLINEPAGE template variable, which can be used to add anchors to
any inline template.
To keep that sufficiently general, it is the full page name, so the
comment anchors and links changed form.
Got rid of the FIXMEd hardcoded html anchor div.
More importantly, the anchor is now to the very top of the comment, not the
text below. So you can see the title, and how it attributes you.
Avoid changing the permalink of pages that are not really comments, but
happen to contain the _comment directive. I think that behavior was a bug,
though not a likely one to occur since _comment should only really be used
on comment pages.
| Joey Hess | 2008-12-18 |
| * | aggregate: If a feed fails to be downloaded, try again immediatly next time a... | Joey Hess | 2008-12-17 |
| * | jump to comments anchor after post•••Not ideal, it would be nicer to jump to the actual comment posted, but no
anchor is available.
| Joey Hess | 2008-12-17 |
| * | typo | Joey Hess | 2008-12-17 |
| * | add Comments link when displaying a page inline•••This link will supplant the usual Discussion link for pages
that have comments enabled.
| Joey Hess | 2008-12-17 |
| * | fix test when comments_closed_pagespec is empty | Joey Hess | 2008-12-17 |
| * | change around comments pagespecs•••I think it is clearer to have one pagespec that controls all pages with
comments, and a separate pagespec that can be used to close new comments on
a subset of those pages.
| Joey Hess | 2008-12-17 |
| * | fix default values for config settings•••defaults cannot be set in getsetup, do it in checkconfig to avoid
uninitialized value warnings.
| Joey Hess | 2008-12-17 |
| * | Coding style change: Remove explcit vim folding markers. | Joey Hess | 2008-12-17 |
| * | checksessionexpiry: rework•••This function as factored out was a bit confusing, I think this makes more
sense.
| Joey Hess | 2008-12-17 |
| * | Merge branch 'master' into comments | Joey Hess | 2008-12-17 |
| |\ |
|
| | * | monotone: When getting the log, tell monotone how many entries we want, rathe... | Joey Hess | 2008-12-17 |
| | * | rename: Fix double-escaping of page name in edit box.•••titlepage normally escapes, but so does formbuilder.
| Joey Hess | 2008-12-17 |
| | * | htmlbalance: don't compact whitespace, and set misc other options•••Not compacting whitespace is the most important one: now that we run
sanitize hooks on individual posted comments in the comments plugin,
whitespace that is significant to Markdown (but not HTML) is lost.
(cherry picked from commit cb5aaa3cee8b35d6fc6e88a7449a9477a6587c7a)
| Simon McVittie | 2008-12-12 |
| * | | elide unnecessary variables | Joey Hess | 2008-12-12 |
| * | | remove fixme•••sessioncgi hooks are always called with the wiki locked
| Joey Hess | 2008-12-12 |
| * | | fix redefinition of $author | Joey Hess | 2008-12-12 |
| * | | move related code together | Joey Hess | 2008-12-12 |
| * | | whitespace | Joey Hess | 2008-12-12 |