• last updated 13 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Return a fallback attachments url when not in a connection context, for example, during mail sending on the bulk-mail package

Add new private proc 'attachments::get_attachments_url', and use it to complement the logic of 'attachments::get_url'

As 'attachments::get_url' returns the value of the attachments package

'RelativeUrl' parameter, which can change at any time, it could happen

that previously mounted attachments have a different url and are not

found anymore.

We try our best here to find a mounted attachments package under

'base_url' to mitigate this, probably flawed, package logic.

In the future, probably a better method should be used for URL resolving

that is not so broken.

The whole thing is even more weird, as the attachments package is

currently a singleton that auto-mounts on /attachments, so i am tempted

to replace this whole thing with just that, but anyway...

Add 'attachments::get_title' to centralize the logic of title guessing for an attachment

Make 'attachments::get_all_attachments' more general, so we try our best to get the 'label' of the attachment by trying different options depending on the object type: 'content_item', 'content_revision', a subtype of those, or just the 'title' row in the 'acs_objects' table. In the very worst case, just set the given 'item_id' as the label.

Prefer live to latest revision when getting title for attachments in 'attachments::get_all_attachments'

When the attachment is not in the file storage, consider both the possibilities of it being a 'content_item' or a 'content_revision', while trying to resolve its title in 'attachments::get_all_attachments' by looking directly in the content respository

Use 'item_id' instead of 'revision_id' for attachments

Do not assume all attachments are in the file storage while retrieving their title/name

Whitespace cleanup

Provide a minimal documentation to public procs

Let everything happen in the database for this proc

  1. … 1 more file in changeset.
Remove divergency between oracle and postgres codebase, rewrite proc to avoid duplication and give it a proper and correct documentation

Quoting commit that replaced db_list_of_lists with db_list, de facto hiding every additional column from results:

-------

Author: Peter Marklund <peter_marklund@fastmail.fm>

Date: Mon Nov 24 14:50:34 2003 +0000

changing the get_attachments procs to use db_list instead of db_list_of_lists and the PG query to only return the id

-------

  1. … 2 more files in changeset.
merged changes from the oacs-5-9 branch and resolved conflicts

  1. … 7834 more files in changeset.
- add editor hints to keep spaces/tabs in the future more consistent

  1. … 313 more files in changeset.
Merging back to HEAD branch oacs-5-8 (using tag vg-merge-oacs-5-8-from-20141027).

  1. … 2547 more files in changeset.
- breace expressions

- used compiled instructions

- bump version number

  1. … 8 more files in changeset.
1. Use the mapped folder from file-storage to get and store attachments

2. HTML cleanup

  1. … 2 more files in changeset.
use content::extlink instead of the deprecated content_extlink:: api, add content::extlink::edit function

  1. … 12 more files in changeset.
1. Merge 2.4.1 to HEAD other than assessment, imsld, lorsm, and views.

2. Bump version numbers to 2.5.0d1.

  1. … 379 more files in changeset.
Fixed problem with nabvar when name contain spaces.

Merge from oacs-5-2 new detach user interface.

  1. … 3 more files in changeset.
Add return_url so you can specify where to go after attaching.

  1. … 3 more files in changeset.
First shot at merging oacs-5-2 apps used in .LRN to HEAD.

  1. … 2478 more files in changeset.
Applied #721, fixed #2563

  1. … 2 more files in changeset.
changing the get_attachments procs to use db_list instead of db_list_of_lists and the PG query to only return the id. Previously committed by mistake on head

  1. … 1 more file in changeset.
changing the get_attachments procs to use db_list instead of db_list_of_lists and the PG query to only return the id

  1. … 1 more file in changeset.
Merged 4.6.4d1 (unreleased) to HEAD. This marks the end of the life of the oacs-4-6 branch.

  1. … 816 more files in changeset.
make file attachments work (my last commit fixed URLs but overlooked files

  1. … 5 more files in changeset.
I18N: made the attachments package use the message catalog

  1. … 13 more files in changeset.
cleanup javadoc tags (eg @cvs_id @cvs @cvs-id: instead if @cvs-id)

  1. … 692 more files in changeset.