| Commit message (Collapse) | Author | Age |
| |
|
|
|
|
|
|
|
|
| |
Previously, prune("wiki/srcdir/sandbox/test.mdwn") could delete srcdir
or even wiki, if they happened to be empty. This is rarely what you
want: there's usually some base directory (destdir, srcdir, transientdir
or another subdirectory of wikistatedir) beyond which you do not want to
delete.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Left out confirmation of removal for held attachments because
a) they're not in the wiki yet, so confirmation is a bit unnecessary
b) it would be hard
c) eases later integration of jquery file upload interface
Also changed where attachments of index are held (to match where they're
stored in the srcdir).
Note that the attachment formbuilder hook was made to run last, so that
the list of attachments is not generated before removal, in the fast path
w/o confirm.
|
| |
|
| |
|
| |
|
|
|
|
| |
These return codes are not currently used, but might be later.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Using named parameters for these is overdue. Passing the session in a
parameter instead of passing username and IP separately will later allow
storing other session info, like username or part of the email.
Note that these functions are not part of the exported API,
and the prototype change will catch (most) skew, so I am not changing
API versions. Any third-party plugins that call them will need updated
though.
|
|
|
|
|
|
|
|
|
|
|
| |
Everywhere that REMOTE_ADDR was used, a session object is available, so
instead use its remote_addr method.
In IkiWiki::Receive, stop setting a dummy REMOTE_ADDR.
Note that it's possible for a session cookie to be obtained using one IP
address, and then used from another IP. In this case, the first IP will now
be used. I think that should be ok.
|
| |
|
|
|
|
|
|
| |
Since all forms are wrapped in a template that defines the actual
stylesheets, formbuilder just has to be told to turn on stylesheet mode,
not what file is the style sheet.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: intrigeri <intrigeri@boum.org>
|
|
|
|
|
|
| |
check_canremove/canrename is called only for its side effect (of failing if
removal is not allowed), its return value is never used and returning
something makes that unclear
|
|\ |
|
| |
| |
| |
| | |
The form was misdisplayed when displayed via comment removal.
|
|/
|
|
| |
Signed-off-by: intrigeri <intrigeri@boum.org>
|
| |
|
| |
|
|
|
|
|
| |
Wired up check_canedit and check_canremove, still need to deal with
check_canattach, and test.
|
|
|
|
|
| |
This avoids another one of those $_ scoping issues where a deep call to a
function that changes $_ clobbers the array that is being looped over.
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
IkiWiki/Plugin/recentchanges.pm
Note that smcv's approach of using urlto also gets the url right when
redirecting to a non-html file, which is a better approach than my recent
fix to recentchanges
|
| | |
|
|/
|
|
| |
should be treated as an attachment.
|
|
|
|
| |
(brain.. melting..)
|
| |
|
|
|
|
|
|
|
|
|
| |
Implemented for git and svn so far.
Note that rcs_commit_staged does assume that the rcs has the ability to
"stage" multiple changes for a later commit. Support for this varies, but
all we really care about is staging removals and renames, which, AFAIK, all
modern rcs's support.
|
| |
|
|
|
|
| |
Break out into function so it can be done up-front before confirmation too.
|
|
|
|
| |
As noted, this may be overkill..
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Including a check the the file is in the srcdir, as opposed to some other
directory.
|
| |
|