Skip to content

Comments

Change: start moving groups section out of manage_sql.c#2658

Merged
greenbonebot merged 4 commits intomainfrom
groups-files-1
Dec 1, 2025
Merged

Change: start moving groups section out of manage_sql.c#2658
greenbonebot merged 4 commits intomainfrom
groups-files-1

Conversation

@mattmundell
Copy link
Contributor

What

Move a few functions from the groups section in manage_sql.c to dedicated groups files.

Why

To reduce the size of manage_sql.c and to better organise the GMP resources.

References

Similar to #2523 for assets.

Testing

I ran GET_GROUPS with the PR and with main, and compared the output:

o m m '<get_groups/>' > /tmp/g-pr
o m m '<get_groups trash="1"/>' > /tmp/g-pr-tr
o m m '<get_groups/>' > /tmp/g-main
o m m '<get_groups trash="1"/>' > /tmp/g-main-tr
diff /tmp/g-main /tmp/g-pr
diff /tmp/g-main-tr /tmp/g-pr-tr

@mattmundell mattmundell requested review from a team as code owners November 28, 2025 09:50
@greenbonebot greenbonebot enabled auto-merge (rebase) November 28, 2025 09:50
@greenbonebot greenbonebot merged commit 7d81b30 into main Dec 1, 2025
28 of 30 checks passed
@greenbonebot greenbonebot deleted the groups-files-1 branch December 1, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants