Extend Python resources API.
Needs ReviewPublic

Authored by woltherav on Jan 14 2018, 3:27 PM.

Details

Reviewers
rempt
Summary

This extends the python resources api with objects for Palette/Brushtip/BrushPreset/Gradient/Pattern. It adds one single notified to Notified.cpp and puts save into resources.cpp.

Part of T7487

Test Plan

Each new resource has a mini python script in the header to test with.

Diff Detail

Repository
R37 Krita
Lint
Lint Skipped
Unit
Unit Tests Skipped
woltherav requested review of this revision.Jan 14 2018, 3:27 PM
woltherav created this revision.
woltherav updated this revision to Diff 25331.Jan 14 2018, 5:36 PM

Added extra stuff for the Notifier and fixed some stuff in the palette.

Two big problems:

  1. I am not sure how the notifier is supossed to work. I cannot get it to work for python.
  2. I am unable to create resource server adapters for brushtips and presets, meaning those signals cannot be sent.

Hm, I guess the first thing I'll do is be to write a unittest for the notifier, so we can be sure it works at all... The resource servers for brush tips and presets are different from the ordinary ones, but I'm going to have to dig in to see what's up with that.

rempt added a comment.Jan 16 2018, 3:16 PM

Okay, I've done the unittest and found the bug in the Notifier... That part should be fine now :-)