3381 Commits (51509e6b34c4640edb1c068ac52f4920edd143e6)

Author	SHA1	Message	Date
Oliver Günther	0883cf9834	Replace exclusive-edit with transcluded comment directive ... This commit removes the somewhat superfluous exlusiveEditController in favor of event-based passing of quoted text to the correct comment field. The comment directive now decides for itself whether it wants to be placed above or below the activities and uses transclusion for that purpose. Editing comments is now no longer exlusive, but that shouldn't be necessary anyway.	9 years ago
Oliver Günther	3bfad28db0	Correct comment behavior with optional notifications ... This allows submitting comments with and without notification for the created journal. Using hyperagent resource on activities, the resource `addComment` is replaced when executing `fetch` on the link. This breaks subsequent comment requests. This is fixed by requesting manually on the resource link using `$http`. This commit also provides the functionality required to comment with or without notification propagation, and changes the backend to comply to the APIv3 specification, which defines comments to be sent as a JSON `{ comment: { raw: 'my comment' } }`. This commit also provides: * Displaying the comment input on overview depending on sort setting * Disable inplace controls when comment is empty * correctly hiding the comment based on link availability	9 years ago
Oliver Günther	e41b8aafee	Add rake task to identify unassociated repositories ... This commit adds a rake task to identify and list repositories in the current managed path that do not have an associated project. [ci skip]	9 years ago
Markus Kahl	ac8a096488	polish considering remarks through review and houndci	9 years ago
Markus Kahl	3a3ff0b102	actually unsubscribe cleared subscriptions	9 years ago
Markus Kahl	45153a5d1b	dummy LDAP auth source ... I don't care if we have a company LDAP. My thoughts go out to devs* who don't have that. *OpenSource, right?	9 years ago
Markus Kahl	2ce62e3999	user invitation polish	9 years ago
Markus Kahl	43e18dd00c	switch to user invitation by default ... this changes quite many flows. With this user self registration is a separate flow to what happens when an admin creates a user. Users created by admins are 'invited' instead of 'registered'.	9 years ago
Oliver Günther	fecd3b3b3e	Simple API for managing the homescreen content ... This commit alleviates the homescreen management from both core and plugins, in order to simply add and remove blocks. Also contains refactoring from review * Consistent block and variable usage * Removed unused labels * Avoid word breaks in multiline enumerations and remove margin from enumerations	9 years ago
Jan Sandbrink	4ee163393f	Rearrange mail delivery responsibilities ... - each delivery job delivers only a single mail (like in the past) - a common super-class ensures correct handling of different error conditions (similar to the past) This new structure should allow for an efficient error handling: - fail the mail delivery on a per-recipient basis (i.e. redeliveries on error only affect the user that did not receive a mail) - only fail the delayed job for delivery errors (e.g. network issues, etc) - do not fail the delayed job for rendering errors (which are likely to re-occur on each rendering) -> only log those	9 years ago
Jan Sandbrink	0702683116	Fix bad method call for version custom fields ... exploring the down sides of isolated unit testing :(	9 years ago
Jan Sandbrink	338f9d40d0	Fix (untested) relation representer ... This one still is not very APIv3, but there ain't no time to improve it... The angular frontend also seems to rely on the quirks with the relation type changing depending which work package embeds the relation.	9 years ago
Jan Sandbrink	88ed4d2df9	Promote ::create to a common interface method ... This one is already used by the representers that require a custom field injector. However, by having a separate method this also means collections (or the new DecoratorFactory) need to take care and call the ::create method instead of ::new. I therefore made create the default interface to create representers, which is now used by Factories and Collections. Note: I did not yet change all representer instantiations to ::create. I hope this will happen gradually.	9 years ago
Jan Sandbrink	6e9ed008fb	Enforce passing of current_user into representers ... Thereby permission checks will no longer silently fail if no user was passed. This uncovered a lot of places where we actually expected a user to be passed, when it wasn't. This was especially true for collections. The basic idea is, that virtually all representers could change their behaviour based on the current user. Representers where this is not the case (e.g. Digest), can still explicitly opt-out.	9 years ago
Jan Sandbrink	f121e610db	Pass current_user into all AttachmentRepresenters	9 years ago
Jan Sandbrink	3f269e43ae	Add link to delete an attachment to its representer	9 years ago
Jan Sandbrink	e7a6ae629e	Add specs for update links with missing permissions ... - also (for now) remove title text which was not localized - consistently use link helpers for the corresponding specs	9 years ago
Jan Sandbrink	aad39544f5	Hide addAttachment link when user is not allowed	9 years ago
Jan Sandbrink	5de0f15898	Correctly link activities ... - they might not be embedded - create an endpoint for activities - document it properly - adapt angular client	9 years ago
Jan Sandbrink	14a55b82d1	Do not embed linked resources in WP collections	9 years ago
Jan Sandbrink	5d473e9a9c	fix HTTP response of DELETE attachment ... this was wrongfully HTTP 202 (Accepted), which would only be valid if we did the deletion in a delayed/deferred way. For now using HTTP 204 (No Content), since we have no response body.	9 years ago
Jens Ulferts	db3681661d	fixes attachment download path in api ... Kudos to @oliverguenter for the fix	9 years ago
Jan Sandbrink	f9cb083481	in production handle internal API errors gracefully ... - still allow for useful errors during development - let all own exceptions inherit from StandardError, the thing you should use for own errors...	9 years ago
Oliver Günther	5b147e32d5	Storage performance and accuracy improvements ... This commit allows storage counting to shell out to `du` where available. The method uses du with `-bs` to count the apparent file size in bytes. Note that apparent size is NOT the same as the real required size on file system, but is used to be consistent with other sizes displayed in OpenProject (e.g., attachments size) When du is either unavailable or incompatible, in that it doesn't return the requested count, the ruby fallback counting method is used. This method has been slightly altered to use `File.stat` instead of `File.new(path).size` for reduced computational overhead compared to `du`.	9 years ago
Henriette Dinger	87c17a0809	made style changes to please rubocop	9 years ago
Henriette Dinger	bca793ce73	replaced images in Repository, Wiki and fixed occurred bugs	9 years ago
Henriette Dinger	fa40630b95	replaced images in Wiki, Tables, TopMenu and restructered the _tables.sass	9 years ago
Alexander Bach	83643c37d2	Remove project homepage field from API	9 years ago
Jan Sandbrink	d8ec635e9e	simplify interface for assignable values ... - one method to rule them all - eases overriding -> does not require to define method for new types on base class - allows to infer values_callback from property name (schema representer)	9 years ago
Jan Sandbrink	9ac9524d28	change attribute registration via plugin to be more generic ... - now also registers the attribtue as writable in the schema	9 years ago
Jan Sandbrink	5540f381a5	adapt acts as op engine to define methods via def ... - putting instance methods into InstanceMethods module is consistent with other ActsAs* implementations - the InstanceMethodsModule also allows me to reduce the diff size, because I keep the indentation of before	9 years ago
Jan Sandbrink	819b69fd66	move auth check out of before block ... different endpoints under work_packages require different permissions	9 years ago
Jan Sandbrink	9e8d051027	reintroduce form_embedded argument for schema ... we can't live without it :/	9 years ago
Jan Sandbrink	2a8659d0f4	split parsing and applying of filters/sort	9 years ago
Jan Sandbrink	cc45136701	refactor 404 auth error ... no separate method needed, let authorize_by_policy behave like well known auth methods (accepting a block)	9 years ago
Jan Sandbrink	32e638449b	hide lockVersion from work package columns	9 years ago
Jan Sandbrink	8e4321959e	improve code style	9 years ago
Jan Sandbrink	781aac5a20	fix stupid C&P error	9 years ago
Oliver Günther	f982465a5d	Force tmpdir for storage spec ... We observed `lib/open_project/storage_spec.rb` to fail occasionally, which I assume has something to do with the existence of the attachments directory. The changed spec forces a tmpdir to be created and evaluates storage information on that path.	9 years ago
Jan Sandbrink	637fadf602	add endpoint to list work package columns in a project	9 years ago
Jan Sandbrink	bfcc769cfd	split implementations for work package schema ... - was already quite case-ish, depending on its creation via WP or via project + type - I plan on adding a third case, which would not improve the situation for a single class	9 years ago
Oliver Günther	9f26c74a38	Require find to avoid LoadError on qa-dev	9 years ago
Alexander Bach	4105a920b5	Style watcher notification mail	9 years ago
Alexander Bach	c22c41b7db	Directly notify new watchers by mail	9 years ago
Oliver Günther	ef322e89dd	Cleanup	9 years ago
Oliver Günther	dfb8ac4853	Add basic checkout instructions ... This commit adds very basic checkout instructions similar to `openproject-checkout` to repositories. Issues: * Uses table style for checkout component, as exact styling is unclear * Adds compressed table style with less padding * Checkout settings are best organized in a Hash with keys per SCM * vendor. Using default settings, a lot of new setting values would be introduced * Checks for Hash in SettingsController * Currently can't use SettingsHelper, as it requires single setting values	9 years ago
Oliver Günther	4707096281	Return nil from storage exception block	9 years ago
Jan Sandbrink	d5fe64bd5e	move writability decision into WP schema ... - reducing the method count (aka interface size) of the schema - giving it more control - still allow outside override in representer (for plugin compatibility)	9 years ago
Jan Sandbrink	0a73e6bac4	convert attribute names in queries API ... This should make the API more consistent again, accepting and producing the same attribute names on all endpoints	9 years ago
Martin Linkhorst	eba94776ae	perm might be nil. in that case just ignore it.	9 years ago