openproject

Commit Graph

Author	SHA1	Message	Date
Oliver Günther	a92575e3ec	Use distinct(*) on aggregated journal to avoid duplicate raw journals https://community.openproject.com/wp/33744	4 years ago
Oliver Günther	c6b9c8ff02	Use distinct(*) on aggregated journal to avoid duplicate raw journals https://community.openproject.com/wp/33744	4 years ago
ulferts	e14679d0b3	avoid loading aggregated journals repeatedly when patching	4 years ago
ulferts	7f39a3b08f	allow fetching activities by an id referencing a subsumed journal AggregatedJournals might contain a set of native journals. This change allows fetching an aggregated journal the id of any journal included in the aggregated. This provided clients with more available references to an aggregated journal. This is particularly helpfull in case the aggregated journal includes more journals than the client expects. This might happen when the journable is updated in the meantime. E.g. a user adds a screenshot to the last comment within the time frame available for aggregation; a user adds a comment, alters an attribute and wants to change the comment. At the same time, this change limits the aggregated journals created in the database to a particular journable. This has a severe performance impact as without the focus on a particular journable, potentially millions of journals will be aggregated to then only fetch one individual result.	4 years ago
ulferts	983709db89	rewrite acts as journalized to create journals in sql	5 years ago
ulferts	79409370ee	ensure thread safety of journal notifications	5 years ago
ulferts	6cdedab3d2	fix typo	5 years ago
ulferts	a6689a9006	let first notification setting dominate child setting	5 years ago
ulferts	2ed7a69943	rely on caller to reset the journal notification setting By forcing the caller to disable mail sending via a block, we can also ensure that the value is reset. But by it, we can allso allow disabling mail sending for multiple save_hook runs which was not possible before.	5 years ago
ulferts	67ade5002d	extract journal notification setting to separate class	5 years ago
ulferts	64d0f57d85	Feature/aggregated activities (#8221 ) * use cte for aggregated journal * Revert "use cte for aggregated journal" This reverts commit `5fedefefdd`. * add another subselect that could later on be provided from the outside * allow passing a nukleous sql to aggregated journals * wip - using aggregated journal for activity * new sql for aggregated journals * start implementing new aggregated query * additional documentation * consolidate activity functionality * simplify by turing into instance methods * move activity fetcher out of redmine * remove verb verification made obsolete Without catchall routes, the dispatching handles it * remove duplicate authorize check * refactor activities controller * refactory activity fetcher * cache avatar file * sort choosable events * remove legacy spec covered by contemporary * speed up aggregated journals via CTE * instance var might never have been set * ensure the event_types are always transmitted * correctly reset the avatar cache * fix avatar fetcher expectation regarding wiki pages * adapt spec [ci skip]	5 years ago
Oliver Günther	53f2d694ff	Use AnyFixtures to generate fixtures from factories (#7230 ) Uses FactoryBot to keep and maintain specific records in a special transaction that does not get removed after each spec. They automatically are created whenever first hitting them. This makes an excellent time saver for items that are commonly used, such as an admin user account	5 years ago
ulferts	0acff96c09	fix journal noop? check	5 years ago
ulferts	59f907560e	no mail on empty wp update	5 years ago
ulferts	a6e87c9dcb	display retracted message on empty journals	5 years ago
ulferts	9c99bc213f	remove duplicate constant assignment	5 years ago
Henriette Dinger	bd7f4e4814	Update copyright notice	5 years ago
ulferts	d37bdb71f0	extract methods	5 years ago
Wieland Lindenthal	2935a46096	WIP: Spike to display BCF viewpoint snapshots in WP activity tab.	5 years ago
Oliver Günther	4666fbb4e0	[30415] Fix missing project reference in journal https://community.openproject.com/wp/30415	5 years ago
ulferts	9c3206220b	remove mysql specific code throughout the application	5 years ago
Oliver Günther	5de0b9a2d8	Go through brakeman warnings	6 years ago
Oliver Günther	1bbad019fe	Correctly escape aggregated journal notes_id [ci skip]	6 years ago
Oliver Günther	a358d45e5f	Correctly escape aggregated journal notes_id [ci skip]	6 years ago
Oliver Günther	79befe97b3	Bump capybara and rspec	6 years ago
Oliver Günther	86be73236f	Prepare for core integration	6 years ago
Oliver Günther	5b49b33b21	Prepare for core integration	6 years ago
Oliver Günther	352099089c	Prepare for core integration	6 years ago
Jens Ulferts	440f101f7e	enforce order when numbering for journal aggregation Otherwise journals recreated later (e.g. when recreating the initial journal) can end up with wrong numbering	6 years ago
Oliver Günther	4a3d220641	[28185] Fix wiki history and annotate https://community.openproject.com/wp/28185	6 years ago
Oliver Günther	6ef4211e77	Bump copyright to 2018 (#6171 ) [ci skip]	7 years ago
Jens Ulferts	30a7b2c601	remove slow implementation of #column_names	7 years ago
Jens Ulferts	6fdd122534	merge wp tree and relations into typed_dag Instead of having the tree/hierarchy information for work packages and the relations between the work packages separated into two tables all the information is now stored in the relations table. To support this, the typed_dag plugin is used. Notable changes: * relations_to and relations_from are swapped. relations_to has all relations that point to another work package (from_id = work_package.id) and relations_from contains all relations that lead to the work package from another one (to_id = work_package.id). This change is merely in the code. The db information is left untouched. * For follows/precedes to_id and from_id are swapped in the db. Because of that, "follows" is now the canonical name instead of "precedes". This is done so that hierarchy relations and follows relations point into the same direction which allows to easily detect cyclic relationships that would lead to infinit scheduling. * the columns `root_id`, `lft`, `rgt` do no longer exist in the wp table. * the column `relation_type` does no longer exist in the relations table. * For every type of relation a column (integer) does now exist in the relations table (e.g. hierarchy, block, follow) * Separate associations exist for every type of relation into two directions. E.g. Work packages following the current one can be accessed via `work_package.precedes`	7 years ago
Oliver Günther	7cf11bb689	Update copyright notice	8 years ago
Oliver Günther	646d99b704	Prepare OpenProject 6.0.0 release	8 years ago
Jens Ulferts	8065c66256	eager loading associated objects, the dirty way	9 years ago
Jens Ulferts	3734b3a232	limit aggregation to one journable With only .where("#{table_name}.id = ?", notes_id) appended, the DB would aggregated all (from all journables of all types) and then select the one that is necessary. This takes a considerable amount of time. The commit will limit the aggregation to one journalizable.	9 years ago
Jan Sandbrink	26574521d3	Prefill predecessors for results of multiple journables too	9 years ago
Jan Sandbrink	0343e12c83	Prefill predecessor when it is already known Getting the predeccessor of a journal is a common operation. When computing the journals for a single journable, it is easily possible to prefill the predecessor (because we know all the journals). Doing this avoids an N+1 query later on (in case the journals are used to display diffs).	9 years ago
Jan Sandbrink	dfb9a5cd20	remove remaining AR find_by_x finders - most of them were removed by @myabc in the rails4 branch - some where reintroduced due to parallel development to that branch - still some occurences left that will hit custom implementations	9 years ago
Jan Sandbrink	b322fafc25	remove workaround for type conversion in PSQL as of Rails 4 this should no longer be neccessary	9 years ago
Jan Sandbrink	3fe8b33f78	enforce disabled aggregations due to limited time resolution we can't assume that a setting of 0 will implicitly disable aggregation	9 years ago
Jan Sandbrink	d47d6685ed	filter for right journal on SQL level	9 years ago
Jan Sandbrink	f9051bba3e	display diff of containing aggregated journal ... instead of displaying the diff for the requested journal only - this should provide the best forward compatibility of links - even when aggregation timeframes are changed intermittently	9 years ago
Jan Sandbrink	e405605343	fix wrongful notificaiton delivery	9 years ago
Jan Sandbrink	285b7f5437	move on-behalf-delivery into notification mailer	9 years ago
Jan Sandbrink	974523c370	partly address an edge case that can prevent notification delivery "partly" because the current implementation is still "vulnerable" if the succeeding journal gets outdated in the same way as the preceeding one. The edge-case of the edge-case....	9 years ago
Jan Sandbrink	6c1e4dbf4f	send aggregated notifications - only sending the last mail of an aggregation chain - displaying the aggregated state of the change - replaces old classes for mail notifications - old special error handling for missing users is obsoleted, since recipients are determined at jobs runtime - special cases for missing author and journal are considered	9 years ago
Jan Sandbrink	1ff8f15fbd	explicitly define order of groups in PSQL Always remember kids: SQL databases do not guarantee an order if you don't ask for one! That is why PostgreSQL sometimes found a predecessor for an aggregated journal that was not supposed to have one: Based on internal decisions the order of rough groups was not ascending (as expected implicitly) and master and addition would switch places. ALWAYS SPECIFY AN ORDER IF YOU EXPECT ONE!	9 years ago
Jan Sandbrink	a2b952f549	improve exception message on missing journable.id previously an invalid MySQL statement would have been indicated, which is correct but not very helpful. On PSQL this will also lead to a DB lockup (at least on environments known to me)	9 years ago

1 2 3

109 Commits (b483e6cd3592ded02f7cbff5ea79666f9ab2247f)