openproject

Commit Graph

Author	SHA1	Message	Date
Oliver Günther	9256c9b46c	Allow to disable ldap groups sync with env flag (#9211 )	4 years ago
ulferts	ce5e06dde2	Fix/non matching pagination (#9202 ) * fix duplicates returned for wp list on multiple matching attachments/comments LEFT JOIN leads to potentially having multiple records returned if a search string is present in more than one attachment/comment. This is not a problem if we fetch work packages directly out of the query results but becomes problematic if only the id is fetched and then used to get the work packages by a subquery. * attachment filters no longer differentiate on tsv_where concatenation As EXISTS and NOT EXISTS is now used, the full string is always to be searched for so `and` concatenation is now always employed. * adapt spec Work packages that have no attachment at all are now also considered to match the filter if a "doesn`t contain" operator is used * less strict rspec rubocops for feature specs * cleanup code	4 years ago
Oliver Günther	e8a0ead6e4	Fix selector on bim theme seeder	4 years ago
OpenProject Actions CI	8019c635ec	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	76eab2f5f4	Change selector on theme seeder [ci skip]	4 years ago
Oliver Günther	6086f320e8	[37036] Always return user attributes when sync_users enabled When a synchronized group has sync_users enabled, we want to synchronize the user to OpenProject even if the connection doesn't have onthefly_register enabled. But currently, the user attributes (name, mail, etc.) are only returned if onthefly_register is set so the synchronization will never work without it being set on the connnection https://community.openproject.org/wp/37036	4 years ago
Oliver Günther	bee673f6f8	[37037] Fix bim seeder overriding existing theme https://community.openproject.org/wp/37037	4 years ago
Oliver Günther	0d949ed752	[36726] Handle multiselect values when trying to apply filter values (#9182 ) * [36726] Handle multiselect values when trying to apply filter values This will ensure that multi-select CFs will not get their value overridden to a filter value when it already has one of the selected values. https://community.openproject.org/wp/36726 * Add spec to test filter is not applied in boards	4 years ago
OpenProject Actions CI	3f34817f94	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	2d22930bdb	[36746] Unset plain password accessor after saving (#9157 ) * [36746] Unset plain password accessor after saving Otherwise, this password remains and is validated again due to "user.register!" which then fails with an exception https://community.openproject.org/wp/36746 * Don't try to use user.password after saving it now gets cleared * Remove user.password usage in specs The user password attribute is an accessor that now gets deleted after saving so tests can no longer depend on it being present after created from a Factory * Also retain user password in mail handler	4 years ago
OpenProject Actions CI	a75ff8534e	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	cc3ee5a40d	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	c7e3d21a0b	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	8b2112f543	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	e675848681	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	adb3691ab3	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	57034bc7d6	[35744] Use case-insenstive string comparison for action boards (#9112 ) https://community.openproject.com/work_packages/35744	4 years ago
Oliver Günther	6d38de10b3	Fix empty story in backlogs https://sentry.openproject.com/sentry/saas-openproject-prod/issues/12414/	4 years ago
Oliver Günther	dfe575ca9c	[36269] Add "other" to pluralization rules for one (#9102 ) * [36269] Add "other" to pluralization rules for one For languages that do not have "one" as a pluralization rule (japanese, for example), the pluralization rule defined by I18n.js does not return a valid string for count: 1. Since we probably do not want to define custom pluralization rules for all languages, we can fall back to the "other" key if "one" is not defined. https://community.openproject.com/wp/36269 * Explicitly wait before clicking on viewpoint The click happens before the side tree is rendered by xeokit, and not being updated afterwards	4 years ago
Oliver Günther	1d163f93bb	Move retry block outside checkbox selector	4 years ago
Aleix Suau	3569dbe8f0	Timing issue solved with retry_block	4 years ago
ulferts	a64c77c1be	avoid test flickering by having more complex password	4 years ago
ulferts	f0cf90d7e1	Fix the rails cell not using the content_tag helper of ActionView with escaping (#9067 ) * fixes show links for project members * Properly escapes the name. The cells gem does not escape strings unlike rails * Fixes the case statement for links and includes the link for placeholder users * Fix escaping of content_tag in rails cell and add spec for that Co-authored-by: Oliver Günther <mail@oliverguenther.de>	4 years ago
Oliver Günther	ea29493ce7	[36353] Restrict possible select options in cost entries to users (#9053 ) https://community.openproject.com/wp/36353	4 years ago
Travis CI User	5efea5cf81	update locales from crowdin [ci skip]	4 years ago
Travis CI User	9187f43935	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	d1c9c32fa0	Remove ldap group memberships when user is destroyed (#9037 )	4 years ago
Oliver Günther	4980b4fb4a	Allow LDAP filters and groups to individually sync users (#9031 )	4 years ago
Travis CI User	db41c16229	update locales from crowdin [ci skip]	4 years ago
Travis CI User	aba74d6211	update locales from crowdin [ci skip]	4 years ago
Travis CI User	1135cb0ddf	update locales from crowdin [ci skip]	4 years ago
ulferts	049f3cb6f5	avoid static ids in spec	4 years ago
Travis CI User	ec555b0398	update locales from crowdin [ci skip]	4 years ago
Markus Kahl	863d227d4b	hcaptcha support (#9022 )	4 years ago
Travis CI User	635e9fcabe	update locales from crowdin [ci skip]	4 years ago
Travis CI User	3e239df370	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	0b0066d81e	Correct spec for dashboards It was moved to the dashboards module but not properly rewritten	4 years ago
Oliver Günther	05fc4c45ae	Add common op-principal component and service to render The service renders just like avatars for users outside angular as it will also be used for rendering group and placeholder "avatars" and replace the UserAvatar render service.	4 years ago
Travis CI User	f028241b61	update locales from crowdin [ci skip]	4 years ago
Travis CI User	b4dfc50803	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	f4dfd6c6c6	[36238] Extract and fix user references in other objects (#9007 ) * Move replacing invalid references into separate job for principals * Write migration to remove existing invalid custom values and responsible * Fix other specs * Fix other specs * rewrite replacing user in records * consolidate principal deletion * include placeholder users in spec Co-authored-by: ulferts <jens.ulferts@googlemail.com>	4 years ago
Oliver Günther	5f8a9a1868	[35157] Fix backlogs settings validations to disallow task = story (#9009 ) Refactors the settings controller into its own controller to allow checking the parameters before saving. Also removes a JS snippet that is no longer being executed since CSP is enforced. https://community.openproject.com/wp/35157	4 years ago
Aleix Suau	8dd5ebfc9f	Fix/34840 filter out not allowed board lists (#8983 ) * Refactor * Added pageSize=0 to make the calls lighter * Refactor * Added pageSize=0 to make the calls lighter * Safe operator on board.queries * Tests failing * Tests failing * Debugging failing test * fix setting up the tests * continue integration test on board error messages * linting Co-authored-by: ulferts <jens.ulferts@googlemail.com>	4 years ago
Travis CI User	79f5e33924	update locales from crowdin [ci skip]	4 years ago
Travis CI User	2e498cebd9	update locales from crowdin [ci skip]	4 years ago
Travis CI User	860b5c4213	update locales from crowdin [ci skip]	4 years ago
Wieland Lindenthal	a18954b2c9	Placeholder user services and administration (#8944 ) * Adding placeholder user contracts * Adding create, update, and delete services for placeholder users * WIP: Adding Placeholder User contract specs [ci skip] * Extract contract validation into common helper * Add common validation in BaseContract + common example for admin checks * Introduce common ModelContract shared context for validations * WIP: PlaceholderUser controller, i18n, and routes [ci skip] * Placeholder users index page and query - moved all group related scopes from User to Principal to make them also available in PlaceholderUser. * end * Create PlaceholderUser * Feature spec for editing a placeholder user * Manage PlaceholderUser memberships The managment of memberships is pretty similar for User and PlaceholderUser. This commit extacts the similarities and uses them for both. * General partial and show view for PlaceholderUser * Delete obosolete partial * Allow RequireAdminGuard to be used as a module function * Fix I18n for confirmation text * Smaller code improvements * Fix: Syntax for accessing status enums was wrong. * Use UpdateService for updating a placeholder user * Add spec for PlaceholderUsersController * First code improvements after code review. - more improvements to come. * Further code improvements after review ... still more to come * Correct namespace of delete service * Fix: Make placeholder user contract validate * Remove :type attribute from base contract of User and PlaceholerUser ...and add it to the CreateContracts. Also add type validations. Further extract shared examples for placeholder user attribute validation * Refactor: Extract membership hook calls to helper * Fix redirect paths for membership controllers * Specs already present in shared exampels. * Fix duplicates routes for users and placeholder users * Fix user path * Add attribute name and lastname We don't need a writeable check as both are equally writable * Replace more references to tab_edit_user_path * Skip specs for PlaceholderUsers::DeletionService We will tackle that service in a separate PR. * Fix module usage of RequireAdminGuard * Fix group filter for placeholder users * Fix invalid reference to expect_valid * Fix: Fix tabbed edit path for placeholder users * Fix status filtering on users * Linting * Improve generalisation of individual principal filter cell - Check for presence of groups and statuses in order to toggle visibility of their UI element. - Remove groups from placeholder user controller and cell initialization and options * Fix selector on groups assign * Remove using_shared_fixtures Co-authored-by: Oliver Günther <mail@oliverguenther.de>	4 years ago
Oliver Günther	cb5fbb6a72	Use the configured variable for expiry https://github.com/opf/openproject/pull/8941	4 years ago
Oliver Günther	8df966abff	Use the configured variable for expiry (#8985 ) https://github.com/opf/openproject/pull/8941	4 years ago
ulferts	1bdd2ab9ae	safe automatic fixes by rubocop (#8994 )	4 years ago

1 2 3 4 5 ...

2302 Commits (2bd3d3e6b96a9836918fc216e015ed797cafaa1f)