Skip to content

Change: move accessors to dedicated target files#2762

Merged
greenbonebot merged 4 commits intomainfrom
targets-files-3
Feb 12, 2026
Merged

Change: move accessors to dedicated target files#2762
greenbonebot merged 4 commits intomainfrom
targets-files-3

Conversation

@mattmundell
Copy link
Contributor

What

Move target property functions out of manage_sql.c.

Why

Better file organisation. Reduces size of manage_sql.c.

References

Follows /pull/2761

Testing

Ran GET_TASKS and GET_TARGETS with main and PR. Made sure output was the same.

o m m '<get_tasks details="1" task_id="8831937d-0e87-439a-807c-a9a2cf976f6a"/>' > /tmp/task-pr
o m m '<get_tasks details="1"/>' > /tmp/tasks-pr
o m m '<get_targets details="1"/>' > /tmp/targets-pr
o m m '<get_tasks details="1" task_id="8831937d-0e87-439a-807c-a9a2cf976f6a"/>' > /tmp/task-main
o m m '<get_tasks details="1"/>' > /tmp/tasks-main
o m m '<get_targets details="1"/>' > /tmp/targets-main
diff -u /tmp/targets-main /tmp/targets-pr
diff -u /tmp/tasks-main /tmp/tasks-pr
diff -u /tmp/task-main /tmp/task-pr

@mattmundell mattmundell requested review from a team as code owners February 11, 2026 08:20
@greenbonebot greenbonebot enabled auto-merge (rebase) February 11, 2026 08:21
@greenbonebot greenbonebot merged commit 0a334ec into main Feb 12, 2026
31 of 32 checks passed
@greenbonebot greenbonebot deleted the targets-files-3 branch February 12, 2026 07:15
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