openproject

Commit Graph

Author	SHA1	Message	Date
Jan Sandbrink	c9f0fba192	increase readability of wp->hash conversion it might use more lines, but I hope there is a gain in readability	9 years ago
Jan Sandbrink	8897235fa4	rewrite find into where, order and limit	9 years ago
Jan Sandbrink	194f1814e1	always return a valid scope though WorkPackage works like a scope most of the time, it is not a scope e.g. you can't directly call :order on WorkPackage Note: I would have loved to already do it the Rails 4 way and call :all However in Rails 3 :all returns an Array and not a Relation, so we will have to wait for the Rails 4(.1) Upgrade to replace :scoped with :all :( Sorry @myabc	9 years ago
Jan Sandbrink	e51a4005ff	try to increase readability - extract scopedeterminition into an extra method - always use "outer" if statements in index method	9 years ago
Jan Sandbrink	280e333f18	explain why some code exists	9 years ago
Jan Sandbrink	2854584cb2	use more clear scopes for autocompletion lets see if that really makes stuff more understandable... I (personally) found the old definition of "all" to be too narrow. Now there is: scope == "all": I want to search over all WPs scope == "relatable": I want to search over all WPs that are relatable to WPs in the given project (via project_id) no scope: If I specified a project search over the WPs of that one, otherwise search everywhere (like "all")	9 years ago
Jan Sandbrink	45815cb9f3	rename query to scope sounds more understandable to me, but I am biased, I guess...	9 years ago
Alex Coles	851f23712f	Prune commented code in TimelinesController [ci skip] Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	8798fe68ad	Teach Experimental API UsersController to pluck Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	0942d7335b	Add explicit join references in controllers Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	2d9cfd97f4	Remove deprecated Relation#all in controllers Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alexander Bach	ffd81c4886	Add some polish	9 years ago
Markus Kahl	bcd981df5c	consider X-Authentication-Scheme; opt scoped realm	9 years ago
Markus Kahl	d854df5175	suppress browser prompt on authenticate failure	9 years ago
Alexander Bach	1a52cc1292	Suppress the hound	9 years ago
Alex Coles	0c729dae54	Replace deprecated calls to Model.scoped Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alexander Bach	c687c63c71	Fix some project url building with identifiers This commit was powered by @ulferts	9 years ago
Alex Coles	ecfa05fad9	Rename Project.public scope to public_projects Avoid collision with `public` class method. As of Rails 4.1, `public` is explicitly black-listed as a scope name: https://github.com/rails/rails/pull/14582	9 years ago
Alex Coles	51512f1522	Temp. comment includes in API v2 ProjectController `includes().find` appears to have issues with polymorphic associations in Rails 4.0.x. Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	e02eb0181d	Migrate AR finder/query methods in controllers Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	ec1bb39f9b	Fix syntax (w/Rubocop) in (Rails) controllers Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	57618b25ec	Replace dynamic finder usages with #find_by, etc. This patch replaces all dynamic finders, for the sake of consistency, although only some methods are deprecated. See: https://github.com/rails/activerecord-deprecated_finders#active-record-deprecated-finders * Revert some `User#find_by_login` usages in cuke steps accidentally removed in `74228b59`. User Story # 20325 Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Jan Sandbrink	9eb7cfbe9b	actually allow the status to be set while moving apparently the status_id param was set to expect an array of values while being moved in this commit: `d7f399ab8d` Probably because the spec started to expect that when it was moved to be an RSpec test. I am not sure if this was ever a valid thing outside the tests...	9 years ago
Jan Sandbrink	d3ceffddfc	actually allow the status to be set while moving apparently the status_id param was set to expect an array of values while being moved in this commit: `d7f399ab8d` Probably because the spec started to expect that when it was moved to be an RSpec test. I am not sure if this was ever a valid thing outside the tests...	9 years ago
Jan Sandbrink	b44b0cd2f9	remove details on spent time from project overview	9 years ago
Jan Sandbrink	42ec887384	uglify order statement at least this one DOES work...	9 years ago
Jan Sandbrink	0f868f17e2	specify order in the Rails 3 way ascending is the default anyway	9 years ago
Jan Sandbrink	9ed5480298	also accept auto complete requests without query	9 years ago
Jan Sandbrink	799f3b9b52	eliminate instance variable	9 years ago
Jan Sandbrink	b265ad88db	increase readability of wp->hash conversion it might use more lines, but I hope there is a gain in readability	9 years ago
Jan Sandbrink	b524b0724d	rewrite find into where, order and limit	9 years ago
Jan Sandbrink	dd052d5114	always return a valid scope though WorkPackage works like a scope most of the time, it is not a scope e.g. you can't directly call :order on WorkPackage Note: I would have loved to already do it the Rails 4 way and call :all However in Rails 3 :all returns an Array and not a Relation, so we will have to wait for the Rails 4(.1) Upgrade to replace :scoped with :all :( Sorry @myabc	9 years ago
Jan Sandbrink	dc71f37587	try to increase readability - extract scopedeterminition into an extra method - always use "outer" if statements in index method	9 years ago
Jan Sandbrink	4556da741e	explain why some code exists	9 years ago
Jan Sandbrink	d59b54a63c	use more clear scopes for autocompletion lets see if that really makes stuff more understandable... I (personally) found the old definition of "all" to be too narrow. Now there is: scope == "all": I want to search over all WPs scope == "relatable": I want to search over all WPs that are relatable to WPs in the given project (via project_id) no scope: If I specified a project search over the WPs of that one, otherwise search everywhere (like "all")	9 years ago
Jan Sandbrink	75b542725a	rename query to scope sounds more understandable to me, but I am biased, I guess...	9 years ago
Alex Coles	bc5abb34ab	Remove explicit require_dependency of Principal This should no longer be necessary (thanks also to `8edff983`). This effectively reverts `423152fa06` Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alexander Bach	cbe64452a8	Improve initialization readability	10 years ago
Alex Coles	077e42ba06	Make VersionsController#update use PATCH Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alexander Bach	6c08706c22	Improve initialization readability	10 years ago
Alex Coles	2640852fe3	Fix MembersController#autocomplete_for_member routing * make routing less convoluted (as in `18835799`). Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	ae97716087	Make MyController#account use PATCH Now routes form helper-generated request correctly. Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	ab0864062e	Make WikiController#rename use PATCH Now routes form helper-generated request correctly. http://weblog.rubyonrails.org/2012/2/26/edge-rails-patch-is-the-new-primary-http-method-for-updates/ Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Jan Sandbrink	4a69d2505a	replace ternaries returning booleans - foo ? true : false was replaced where foo already was bool - foo ? true : false was kept, when the ternary did a bool conversion - foo ? false : true was replaced by either negation or replacing == with != Note: I intentionally left one occurence that is fixed in another running PR and would cause a conflict	10 years ago
Alex Coles	a464c01203	Force bool coercion in API v2 WorkflowsController Boolean values are not properly coerced when using `.select` with custom SQL String on PostgreSQL. Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	55a2207631	Replace GroupUser join model w/ habtm association Active Record 4.x does not generate correct DELETE SQL statements when deleting groups for a given `User` (the `GroupUser` join model does not have a primary key). Callbacks (e.g. `after_remove`) are invoked on `has_and_belongs_to_many` associations. This effectively reverts `6c5239c4e2` Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Florian Kraft	f78d1b6254	use toolbar component in news sections Signed-off-by: Florian Kraft <f.kraft@finn.de>	10 years ago
Alex Coles	1ee9b8ee98	Update includes syntax in GroupsController queries Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Jan Sandbrink	dcd75d2a52	"fix" too much space before content in Wiki This is a dirty hack to achieve the goal. Nobody should ever have to do this in his live, but there are deadlines and issues considered important... This fixes the issue the dirty way, but it works and has few implications on all other parts of the application. I hope someone will come and delete those lines quickly...	10 years ago
Markus Kahl	d26382362f	jQuery all the way	10 years ago

1 2 3 4 5 ...

2746 Commits (efdfcf82e4ed615ad1d2e2a0693399d35a9b98c7)