Check for attachable subpockets first when holstering

[dseguin]

Summary

None

Purpose of change

The "attachable-subpockets-as-holsters" feature in #56367 assumes the parent item is not defined as a holster, since "real" holsters use a different holstering action. This means that some MOLLE gear won't show attached pockets when holstering:

(there's an unlisted tactical holster attached)

Cataclysm-DDA/data/json/items/armor/belts.json

Lines 325 to 326 in 69830f8

	"use_action": [
	{ "type": "holster", "holster_prompt": "Store tool or blade", "holster_msg": "You put your %1$s in your %2$s" },

Describe the solution

Switch the check around so that attachable subpockets are listed before treating the parent item as a regular holster:

Describe alternatives you've considered

Testing

The unit tests added in #56367 still pass as expected:

./tests/cata_test --rng-seed time "[pocket]"

Additional context