It's more than likely that nobody has uploaded anything yet.

The name is largely irrelevant as it's only the creator of the file.

Firstly, an over-enthusiastic optimisation I was responsible for broke the replacement of downloadtemp images, especially on sites which are not installed at the root of a domain. Then, some greedy regexes meant that if there was more than one image in the post, only one of them would have the URL fixed properly. The same bug was happening again when posts were being displayed in views.

This patch fixes these problems, and contains a blog database upgrade to fix existing broken paths.

Change paths that were referring to view/view.php?id=..&artefact=.. to instead point to view/artefact.php?artefact=..&view=..

This is related to the viewartefactsplit branch. Now the existing code actually uses the new artefact.php script.

Signed-off-by: Andrew McMillan

Changed all requires.
Also removed not used json/renderartefact.php

& updated call to it in artefact/blog

Corrected another mistake I made with renaming the view= parameter to id= when the URL wasn't actually to view/view.php

Made attaching and detaching files add a record in the artefact_parent_cache linking the entire blog to the attachment. This means that attachments can be shown in views instantly.

Renamed the 'view' parameter for view/view.php to be 'id'. 'view' will continue to work for a while.

view/blocks.php uses id, and id is used for other objects in the system, so it doesn't make much sense for this one to remain this way.

Turns out I made a booboo when I said we could delete render_self - it's used on the 'view artefact' page. So it's back, and updated to give improved display of attachments.

It now takes a hash of options rather than just an artefact ID. This allows you to get icons of different sizes, and more importantly, show icons only in the context of certain views.

Now permissions are checked in the blog edit/view pages, rather than in the constructor, which was only later going to end in pain and suffering. This will mean that views with blogs in them will work again, as per bug #1168.

Now, table names (and other identifiers that could be confused with SQL kewords), should be put into SQL like this:

SELECT * FROM {artefact}

The braces are matched and expanded by DML now to include the prefix and be properly quoted, which means that tables like 'view' and 'group' don't need renaming (and nor should they, there's little reason why we should rename tables to get around such constraints).

This has removed a whole bunch of $prefix = ... and get_config('dbprefix') stuff, which makes things a little simpler yet again.

The blog security fixes were _too_ secure - you couldn't post any new blogs or posts at all\! Fixed now

the blog posts from showing duplicate format options

to views

build links to download files passing the correct view ID

* Added get_links to ArtefactType base class
* Implement get_links for all existing artefact types
* Enabled direct folder/file linking in the my files section

broken due to artefact parent cache...

database changes