Changeset View
Standalone View
source/blender/editors/space_image/image_ops.c
| Show First 20 Lines • Show All 1,529 Lines • ▼ Show 20 Lines | RNA_def_boolean(ot->srna, | ||||
| true, | true, | ||||
| "Detect UDIMs", | "Detect UDIMs", | ||||
| "Detect selected UDIM files and load all matching tiles"); | "Detect selected UDIM files and load all matching tiles"); | ||||
| } | } | ||||
| /** \} */ | /** \} */ | ||||
| /* -------------------------------------------------------------------- */ | /* -------------------------------------------------------------------- */ | ||||
| /** \name Browse Image Operator | |||||
| * \{ */ | |||||
| typedef struct ImageBrowseOp { | |||||
| Image *ima; | |||||
| } ImageBrowseOp; | |||||
| static int image_browse_exec(bContext *C, wmOperator *op) | |||||
| { | |||||
| ImageBrowseOp *ibo = op->customdata; | |||||
| if (ibo == NULL) { | |||||
| return OPERATOR_CANCELLED; | |||||
| } | |||||
| char filepath[FILE_MAX]; | |||||
| RNA_string_get(op->ptr, "filepath", filepath); | |||||
| Image *ima = ibo->ima; | |||||
| /* If loading into a tiled texture, ensure that the filename is tokenized. */ | |||||
| if (ima->source == IMA_SRC_TILED) { | |||||
| char *filename = (char *)BLI_path_basename(filepath); | |||||
| BKE_image_ensure_tile_token(filename); | |||||
| } | |||||
| PointerRNA imaptr; | |||||
| PropertyRNA *imaprop; | |||||
| RNA_id_pointer_create(&ima->id, &imaptr); | |||||
| imaprop = RNA_struct_find_property(&imaptr, "filepath"); | |||||
| RNA_property_string_set(&imaptr, imaprop, filepath); | |||||
| RNA_property_update(C, &imaptr, imaprop); | |||||
| MEM_freeN(op->customdata); | |||||
| return OPERATOR_FINISHED; | |||||
| } | |||||
| static void image_browse_cancel(bContext *UNUSED(C), wmOperator *op) | |||||
| { | |||||
| MEM_freeN(op->customdata); | |||||
| op->customdata = NULL; | |||||
| } | |||||
| static int image_browse_invoke(bContext *C, wmOperator *op, const wmEvent *event) | |||||
| { | |||||
| Image *ima = image_from_context(C); | |||||
| if (!ima) { | |||||
| return OPERATOR_CANCELLED; | |||||
| } | |||||
| char filepath[FILE_MAX]; | |||||
| BLI_strncpy(filepath, ima->filepath, sizeof(filepath)); | |||||
| /* Shift+Click to open the file, Alt+Click to browse a folder in the OS's browser. */ | |||||
| if (event->modifier & (KM_SHIFT | KM_ALT)) { | |||||
| wmOperatorType *ot = WM_operatortype_find("WM_OT_path_open", true); | |||||
| PointerRNA props_ptr; | |||||
| if (event->modifier & KM_ALT) { | |||||
| char *lslash = (char *)BLI_path_slash_rfind(filepath); | |||||
| if (lslash) { | |||||
| *lslash = '\0'; | |||||
| } | |||||
| } | |||||
| else if (ima->source == IMA_SRC_TILED) { | |||||
| ImageUser iuser; | |||||
| BKE_imageuser_default(&iuser); | |||||
| /* Use the file associated with the active tile. Otherwise use the first tile. */ | |||||
| const ImageTile *active = (ImageTile *)BLI_findlink(&ima->tiles, ima->active_tile_index); | |||||
| iuser.tile = active ? active->tile_number : ((ImageTile *)ima->tiles.first)->tile_number; | |||||
| BKE_image_user_file_path(&iuser, ima, filepath); | |||||
| } | |||||
| WM_operator_properties_create_ptr(&props_ptr, ot); | |||||
| RNA_string_set(&props_ptr, "filepath", filepath); | |||||
| WM_operator_name_call_ptr(C, ot, WM_OP_EXEC_DEFAULT, &props_ptr, NULL); | |||||
| WM_operator_properties_free(&props_ptr); | |||||
| return OPERATOR_CANCELLED; | |||||
| } | |||||
| ImageBrowseOp *ibo = MEM_callocN(sizeof(ImageBrowseOp), "ImageBrowseOp"); | |||||
| ibo->ima = ima; | |||||
| op->customdata = ibo; | |||||
Severin: This shouldn't be needed. `image_browse_exec()` can just call `image_from_context()` again. The… | |||||
deadpinAuthorUnsubmitted Done Inline ActionsYou would think that's the case, but it appears not so :-/ The context during invoke has the image correctly attached to it, but not during execute, so null is returned. There is a fallback check but that's only for space image data (sima) which doesn't work when this template is used in other editors (like for Image empties or Camera backgrounds). deadpin: You would think that's the case, but it appears not so :-/ The context during invoke has the… | |||||
SeverinUnsubmitted Not Done Inline ActionsApparently the image is passed via the button context (and if that fails through the image editor context, and if that fails, directly queried from the image editor). This button context isn't available anymore when the File Browser calls the exec() callback. It would be nice to support restoring it, like we already restore the window, area & region context, but I'm not at all keen on it since the data passed to the button context might get freed while the File Browser is open. In practice it probably would be fine, at least for IDs, since these are only freed/reallocated over undo/redo, which cancels the file operation anyway. Still not keen about it. So I'd say querying the data in the invoke() and storing it in the custom data is indeed the best way to go for now. /* The image is typically passed to the operator via layout/button context (e.g. * #uiLayoutSetContextPointer()). The File Browser doesn't support restoring this context when * calling `exec()` though, so we have to pass it the image via custom data. */ Lastly, I'd suggest adding a poll() function that checks if image_from_context() succeeds, so it's grayed out in the UI and search menus if not. Severin: Apparently the image is passed via the button context (and if that fails through the image… | |||||
| image_filesel(C, op, filepath); | |||||
| return OPERATOR_RUNNING_MODAL; | |||||
| } | |||||
| void IMAGE_OT_browse(wmOperatorType *ot) | |||||
| { | |||||
| /* identifiers */ | |||||
| ot->name = "Browse Image"; | |||||
| ot->description = | |||||
| "Open an image file browser, hold Shift to open the file, Alt to browse containing " | |||||
| "directory"; | |||||
| ot->idname = "IMAGE_OT_browse"; | |||||
| /* api callbacks */ | |||||
| ot->exec = image_browse_exec; | |||||
| ot->invoke = image_browse_invoke; | |||||
| ot->cancel = image_browse_cancel; | |||||
| /* flags */ | |||||
| ot->flag = OPTYPE_UNDO; | |||||
| /* properties */ | |||||
| WM_operator_properties_filesel(ot, | |||||
| FILE_TYPE_FOLDER | FILE_TYPE_IMAGE | FILE_TYPE_MOVIE, | |||||
| FILE_SPECIAL, | |||||
| FILE_OPENFILE, | |||||
| WM_FILESEL_FILEPATH | WM_FILESEL_RELPATH, | |||||
| FILE_DEFAULTDISPLAY, | |||||
| FILE_SORT_DEFAULT); | |||||
| } | |||||
| /** \} */ | |||||
| /* -------------------------------------------------------------------- */ | |||||
| /** \name Match Movie Length Operator | /** \name Match Movie Length Operator | ||||
| * \{ */ | * \{ */ | ||||
| static int image_match_len_exec(bContext *C, wmOperator *UNUSED(op)) | static int image_match_len_exec(bContext *C, wmOperator *UNUSED(op)) | ||||
| { | { | ||||
| Scene *scene = CTX_data_scene(C); | Scene *scene = CTX_data_scene(C); | ||||
| Image *ima = image_from_context(C); | Image *ima = image_from_context(C); | ||||
| ImageUser *iuser = image_user_from_context(C); | ImageUser *iuser = image_user_from_context(C); | ||||
| ▲ Show 20 Lines • Show All 2,603 Lines • Show Last 20 Lines | |||||
This shouldn't be needed. image_browse_exec() can just call image_from_context() again. The struct and the customdata management can be removed then.