Asset Pipeline: Improve Push Operator UI #199

Merged
2 changed files with 28 additions and 57 deletions

View File

@ -168,17 +168,13 @@ class ASSETPIPE_OT_create_new_asset(bpy.types.Operator):
for obj in bpy.data.objects:
bpy.data.objects.remove(obj)
bpy.ops.outliner.orphans_purge(
do_local_ids=True, do_linked_ids=False, do_recursive=True
)
bpy.ops.outliner.orphans_purge(do_local_ids=True, do_linked_ids=False, do_recursive=True)
def _task_layer_collections_set(self, context, asset_col, local_tls):
for task_layer_key in config.TASK_LAYER_TYPES:
if task_layer_key not in local_tls:
continue
col_name = (
f"{self._name}{constants.NAME_DELIMITER}{task_layer_key}"
).lower()
col_name = (f"{self._name}{constants.NAME_DELIMITER}{task_layer_key}").lower()
bpy.data.collections.new(col_name)
task_layer_col = bpy.data.collections.get(col_name)
task_layer_col.asset_id_owner = task_layer_key
@ -278,7 +274,7 @@ class ASSETPIPE_OT_update_ownership(bpy.types.Operator):
class ASSETPIPE_OT_sync_pull(bpy.types.Operator):
bl_idname = "assetpipe.sync_pull"
bl_label = "Pull from Publish"
bl_label = "Pull Asset"
bl_description = """Pull Task Layers from the published sync target"""
_temp_transfer_data = None
@ -328,8 +324,8 @@ class ASSETPIPE_OT_sync_pull(bpy.types.Operator):
class ASSETPIPE_OT_sync_push(bpy.types.Operator):
bl_idname = "assetpipe.sync_push"
bl_label = "Push from Publish"
bl_description = """Push the current Task Layer to the published sync target"""
bl_label = "Sync Asset"
bl_description = """Sync the current Task Layer to the published sync target. File will be saved as part of the Push process"""
_temp_transfer_data = None
_invalid_objs = []
@ -350,12 +346,6 @@ class ASSETPIPE_OT_sync_push(bpy.types.Operator):
description="Pull in any new data from the Published file before Pushing",
)
save: bpy.props.BoolProperty(
name="Save File & Images",
default=True,
description="Save Current File and Images before Push",
)
@classmethod
def poll(cls, context: bpy.types.Context) -> bool:
if context.mode == 'OBJECT':
@ -368,14 +358,13 @@ class ASSETPIPE_OT_sync_push(bpy.types.Operator):
return context.window_manager.invoke_props_dialog(self, width=400)
def draw(self, context: bpy.types.Context):
prefs = get_addon_prefs()
if prefs.is_advanced_mode:
self.layout.prop(self, "pull")
self.layout.prop(self, "save")
if not self.pull:
col = self.layout.column()
col.label(text="Force Pushing without pulling can cause data loss", icon="ERROR")
col.separator()
sync_draw(self, context)
def execute(self, context: bpy.types.Context):
if self.save:
save_images()
bpy.ops.wm.save_mainfile()
@ -466,9 +455,7 @@ class ASSETPIPE_OT_publish_staged_as_active(bpy.types.Operator):
def execute(self, context: bpy.types.Context):
current_file = Path(bpy.data.filepath)
staged_file = find_latest_publish(
current_file, publish_type=constants.STAGED_PUBLISH_KEY
)
staged_file = find_latest_publish(current_file, publish_type=constants.STAGED_PUBLISH_KEY)
# Delete Staged File
staged_file.unlink()
catalog_id = context.scene.asset_pipeline.asset_catalog_id
@ -479,9 +466,7 @@ class ASSETPIPE_OT_publish_staged_as_active(bpy.types.Operator):
class ASSETPIPE_OT_reset_ownership(bpy.types.Operator):
bl_idname = "assetpipe.reset_ownership"
bl_label = "Reset Ownership"
bl_description = (
"""Reset the Object owner and Transferable Data on selected object(s)"""
)
bl_description = """Reset the Object owner and Transferable Data on selected object(s)"""
@classmethod
def poll(cls, context: bpy.types.Context) -> bool:
@ -510,9 +495,7 @@ class ASSETPIPE_OT_update_local_task_layers(bpy.types.Operator):
@classmethod
def poll(cls, context: bpy.types.Context) -> bool:
asset_pipe = context.scene.asset_pipeline
new_local_tl = [
tl.name for tl in asset_pipe.all_task_layers if tl.is_local == True
]
new_local_tl = [tl.name for tl in asset_pipe.all_task_layers if tl.is_local == True]
local_tl = [tl.name for tl in asset_pipe.local_task_layers]
if new_local_tl == local_tl:
cls.poll_message_set("Local Task Layers already match current selection")
@ -529,9 +512,7 @@ class ASSETPIPE_OT_update_local_task_layers(bpy.types.Operator):
text="Caution, this only affects current file.",
icon="ERROR",
)
layout.label(
text="Two files owning the same task layer can break merge process."
)
layout.label(text="Two files owning the same task layer can break merge process.")
def execute(self, context: bpy.types.Context):
asset_pipe = context.scene.asset_pipeline
@ -544,9 +525,7 @@ class ASSETPIPE_OT_update_local_task_layers(bpy.types.Operator):
class ASSETPIPE_OT_revert_file(bpy.types.Operator):
bl_idname = "assetpipe.revert_file"
bl_label = "Revert File"
bl_description = (
"""Revert File to Pre-Sync State. Revert will not affect Published files"""
)
bl_description = """Revert File to Pre-Sync State. Revert will not affect Published files"""
_temp_file = ""
_source_file = ""
@ -645,9 +624,7 @@ class ASSETPIPE_OT_update_surrendered_transfer_data(bpy.types.Operator):
bl_label = "Claim Surrendered"
bl_description = """Claim Surrended Transferable Data Owner"""
transfer_data_item_name: bpy.props.StringProperty(
name="Transferable Data Item Name"
)
transfer_data_item_name: bpy.props.StringProperty(name="Transferable Data Item Name")
_surrendered_transfer_data = None
_old_onwer = ""
@ -776,9 +753,7 @@ class ASSETPIPE_OT_batch_ownership_change(bpy.types.Operator):
transfer_data_items_to_update = []
if self.data_type == "TRANSFER_DATA":
for obj in objs:
filtered_transfer_data = self._filter_by_name(
context, obj.transfer_data_ownership
)
filtered_transfer_data = self._filter_by_name(context, obj.transfer_data_ownership)
for transfer_data_item in filtered_transfer_data:
if self.transfer_data_type != "NONE":
if transfer_data_item.type == self.transfer_data_type:
@ -790,8 +765,7 @@ class ASSETPIPE_OT_batch_ownership_change(bpy.types.Operator):
return [
item
for item in transfer_data_items_to_update
if item.surrender
and item.owner not in asset_pipe.get_local_task_layers()
if item.surrender and item.owner not in asset_pipe.get_local_task_layers()
]
if self.filter_owners == "LOCAL":
@ -801,17 +775,13 @@ class ASSETPIPE_OT_batch_ownership_change(bpy.types.Operator):
if item.owner in asset_pipe.get_local_task_layers()
]
if self.set_surrender:
return [
item for item in transfer_data_items_to_update if not item.surrender
]
return [item for item in transfer_data_items_to_update if not item.surrender]
return transfer_data_items_to_update
def _get_objects(self, context):
asset_objs = context.scene.asset_pipeline.asset_collection.all_objects
selected_asset_objs = [
obj for obj in asset_objs if obj in context.selected_objects
]
selected_asset_objs = [obj for obj in asset_objs if obj in context.selected_objects]
return asset_objs if self.data_source == "ALL" else selected_asset_objs
def _get_filtered_objects(self, context):
@ -840,11 +810,7 @@ class ASSETPIPE_OT_batch_ownership_change(bpy.types.Operator):
transfer_data_items_to_update = self._get_transfer_data_to_update(context)
data_type_name = "Transferable Data Item(s)"
length = (
len(transfer_data_items_to_update)
if transfer_data_items_to_update
else 0
)
length = len(transfer_data_items_to_update) if transfer_data_items_to_update else 0
if self.claim_surrender:
action = "Claim Surrendered on"
if self.set_surrender:

View File

@ -59,12 +59,17 @@ class ASSETPIPE_PT_sync(bpy.types.Panel):
staged = is_staged_publish(Path(bpy.data.filepath))
sync_target_name = "Staged" if staged else "Active"
layout.operator("assetpipe.sync_push", text=f"Push to {sync_target_name}", icon="TRIA_UP")
layout.operator(
"assetpipe.sync_pull",
text=f"Pull from {sync_target_name}",
icon="TRIA_DOWN",
)
layout.operator(
"assetpipe.sync_push", text=f"Sync from {sync_target_name}", icon="FILE_REFRESH"
).pull = True
layout.operator(
"assetpipe.sync_push", text=f"Force Push to {sync_target_name}", icon="TRIA_UP"
).pull = False
layout.separator()
if staged: