#1971384: [META] Convert page callbacks to controllers
Adding tags.
I believe #1974372: Convert aggregator_page_sources() to a Controller may be needed to complete this.
Is it worth uploading a patch that is independent of the work needed on the controller?
we should probably wait for #2040199: Convert aggregator_page_category_form and aggregator_page_source_form to a Controller to introduce the categorize form and then reuse it
Marking this as a duplicate of #2040199: Convert aggregator_page_category_form and aggregator_page_source_form to a Controller as it contains the conversion for both aggregator_page_source_form() and aggregator_page_category_form().
aggregator_page_source_form()
aggregator_page_category_form()
Comments
Comment #1
trevjs CreditAttribution: trevjs commentedAdding tags.
Comment #2
trevjs CreditAttribution: trevjs commentedI believe #1974372: Convert aggregator_page_sources() to a Controller may be needed to complete this.
Is it worth uploading a patch that is independent of the work needed on the controller?
Comment #3
ParisLiakos CreditAttribution: ParisLiakos commentedwe should probably wait for #2040199: Convert aggregator_page_category_form and aggregator_page_source_form to a Controller to introduce the categorize form and then reuse it
Comment #4
kim.pepperMarking this as a duplicate of #2040199: Convert aggregator_page_category_form and aggregator_page_source_form to a Controller as it contains the conversion for both
aggregator_page_source_form()
andaggregator_page_category_form()
.