OpenProject is the leading open source project management software.

kanbanworkflowstimelinescrumrubyroadmapproject-planningproject-managementopenprojectangularissue-trackerifcgantt-chartganttbug-trackerboardsbcf

You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Jens Ulferts 994c2d469c avoid adding non writeable filters multiple times ... When the create_class method is called on WorkPackagePayloadRepresenter, a subclass of the class is created that includes the relevant custom fields. That class gets the PayloadRepresenter module included. The payload representer adds a render_filter to the attributes to avoid rendering unwriteable attributes. But when adding the filter, it is not added to an instance of the attribute in the subclass (WorkPackagePayloadRepresenter + CustomFields)but rather to the one in the superclass (WorkPackagePayloadRepresenter). The filter list instance is shared by all subclasses. That sideeffect is not dangerous per se as we want all WorkPackagePayloadRepresenters to have the filter. But as we have one instance of an array, adding to that array will lead to having the filter multiple times which will then all have to be evaluated on every rendering pass. By checking if the filter is already included before adding it, we can avoid this behaviour. The better fix would be to rework the way custom fields are handled in the representers. This could be done by having one representer for the static attributes and one for the dynamic custom fields. Those two representers could be rendered/parsed independently and the results could then be merged. This approach however takes considerably more time.		6 years ago
..
api	avoid adding non writeable filters multiple times	6 years ago
constraints	Bump copyright to 2018 (#6171)	7 years ago
generators/open_project/plugin	Bump copyright to 2018 (#6171)	7 years ago
open_project	Bumped version to 8.0.1	6 years ago
pagination	Bump copyright to 2018 (#6171)	7 years ago
plugins	remove code duplication	6 years ago
redmine	Ensure wiki menu items get created	6 years ago
services	Bump copyright to 2018 (#6171)	7 years ago
tasks	[28517] Fix moving attachment when its the same file	6 years ago
ar_condition.rb	Bump copyright to 2018 (#6171)	7 years ago
chili_project.rb	Bump copyright to 2018 (#6171)	7 years ago
copy_model.rb	Bump copyright to 2018 (#6171)	7 years ago
custom_field_form_builder.rb	[28155] Replace calendar_for with global augmented datepickers	6 years ago
deprecated_alias.rb	Bump copyright to 2018 (#6171)	7 years ago
diff.rb	Bump copyright to 2018 (#6171)	7 years ago
extended_http.rb	Bump copyright to 2018 (#6171)	7 years ago
instance_finder.rb	Bump copyright to 2018 (#6171)	7 years ago
open_project.rb	remove no longer required gems	7 years ago
pagination.rb	Bump copyright to 2018 (#6171)	7 years ago
redmine_plugin_locator.rb	Bump copyright to 2018 (#6171)	7 years ago
scenario_disabler.rb	Bump copyright to 2018 (#6171)	7 years ago
tabular_form_builder.rb	Move ckeditor components into one and add markdown source	6 years ago