Skip entries update and bugfix

[bellerbrock]

Description

Provide a summary of your changes including motivation and context. If these changes fix a bug or resolves a feature request, be sure to link to that issue.

Fixes bug where skip entries for all traits preference would crash collect activity by attempting to set traitboxview to a trait that wasn't visible.

closes #987

Type of change

What type of changes does your code introduce? Put an x in boxes that apply.

• [ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
• [ ] FEATURE (non-breaking change which adds functionality)
• [ ] BUGFIX (non-breaking change which fixes an issue)
• [ ] ENHANCEMENT (non-breaking change which improves existing functionality)
• [ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Checklist:

• [ ] I have performed a self-review of my code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] I have updated relevant documentation

Changelog entry

Please add a one-line changelog entry below. This will be copied to the changelog file during the release process.

Skip entries update and bugfix

[bellerbrock]

@chaneylc thanks. And agreed re the setting description. They have been updated in the v6 settings PR to ` Skip Entries

Disabled

<string name="preferences_behavior_skip_entries_with_data">Arrows skip entries that already have values for the current trait</string>
<string name="preferences_behavior_skip_entries_across_all_traits">Arrows skip entries that already have values for all traits</string>`