[GUI Plugin] Action Chains

[un_pogaz]

Originally Posted by ownedbycats

Some of the actions have conditionals (for example, run Extract ISBN only if book has no existing ISBN identifier). It doesn't quite work right when some book matches conditionals and others don't.

Yeah, I enconter the same problem. It's really confusing to see that the condition depends exclusively on the exact book wich on you open the Action Chains context menu (or the last/first selected) to check the execution conditions, instead of acting as a filter.
If this book matches the condition, the action will be performed on all selected books, even those that do not match and on which you did not want to perform this action.

And I don't think a real filter feature would be possible, because Action Chains often uses the selected books to retrive the id to perform their own action, so it real not depending of Action Chains that the Calibre actions are all or nothing. A real filter feature would involve deselecting and selecting dynamically wich rows is needed before run the Calibre actions, then redo that at each step of the chain.
... I think it's possible (Calibre is powerfull like that), but it would be terribly complicated, so I will give it no hope.

[un_pogaz]

Okay. By checking the code, I found a possible way.

`gui.current_view().get_selected_ids()` allow to retrive the book id of the selected row, and `gui.current_view().select_rows()` allow to select the book base on their id `class calibre.gui2.library.views.BooksView. Additionaly, using this two function allow to mimic and preserve the selection order.

Beside @capink, get_selected_ids() it probably a better ways to retrive the book id than you current way.
Damn it, if you had GitHub or similiar, I could work on proposing a patch, to either transform the condition on a true filter, or add a new filter feature.
I'll see what I can do, but it won't be as clean to share you the full files like that.

[capink]

Originally Posted by ownedbycats

Some of the actions have conditionals (for example, run Extract ISBN only if book has no existing ISBN identifier). It doesn't quite work right when some book matches conditionals and others don't.

Originally Posted by un_pogaz

I could work on proposing a patch, to either transform the condition on a true filter, or add a new filter feature.
I'll see what I can do, but it won't be as clean to share you the full files like that.

This issue and the solution for it has been addressed before. As you can see in the linked post, you can easily modify the selection (into a subset of the already selected books) based on whatever criteria you want. Conditions were intended as a quick way to determine whether a chain/action will run based on simple criteria that does not take much time to run, because chain conditions are executed every time the Action Chains menu is dynamically created. Any more additional criteria can be done as illustrated in the linked post.

The solution in the post even allows for saving the initial selection into a variable, so that you can restore them mid-chain if you want, and allow for more flexible control.

[un_pogaz]

Interesing, but I think that a formal implementation of a filter system would be a big plus, certainly with its own column/settings.
Also, if I understand your idea of wanting to dynamically display the menu entry based on simple condition criteria and so limit this to a single metadata object for performance reason, I think that inside the chains themselves, this is too confusive. It's certainly too late to replace this column with a proper filter setting, but in any case, I really encourage the addition of such thing.

Else, I worked of the idea and so here my proposal:

Spoiler:

in class chains.Chain._run_loop()

somewhere at the start

# store selected books at the start of the chains
self.chain_selected_ids = self.gui.current_view().get_selected_ids()
self.previous_selected_ids = self.chain_selected_ids

after

do_run = self.check_action_conditions(chain_link.get('condition_settings', {}))

append

# filtre the book for running action
filter_settings = chain_link.get('filter_settings', {})
if not filter_settings.get('template'):
    # No conditions set, use all selected ids
    action_selected_ids = self.chain_selected_ids
else:
    action_selected_ids = []
    for book_id in self.chain_selected_ids:
        mi = self.db.new_api.get_proxy_metadata(book_id)
        if self.check_conditions(filter_settings, mi=mi):
            action_selected_ids.append(book_id)

# action_selected_ids can only contain a subset of chain_selected_ids
# don't perform a new select_rows() if the previous seletion was the same than chain_selected_ids
if len(action_selected_ids) != len(self.chain_selected_ids) or len(self.previous_selected_ids) != len(self.chain_selected_ids):
    # change_current=True because the previous current_row can probaly not match the conditions
    # and so the validate() function can fail and abort the action
    self.gui.current_view().select_rows(action_selected_ids, change_current=True, scroll=False)
self.previous_selected_ids = action_selected_ids

and to insert in the finaly block

# full reset seleted books
if len(self.previous_selected_ids) != len(self.chain_selected_ids):
    self.gui.current_view().select_rows(self.chain_selected_ids, change_current=True, scroll=False)

[chaley]

Originally Posted by un_pogaz

Else, I worked of the idea and so here my proposal:
[/SPOILER]

FWIW: I'm currently implementing these changes to the base formatter.

• ids = selected_books()
• name = selected_cell(). Returns the lookup name.
• A 'with' statement that changes the book the template is using for the statement's block. The same local variables are available to the inner and outer block.

Example:

program: 
	ids = selected_books();
	col = selected_column();
	res = '';
	for id in ids:
		with id:
			# prints information from the book with identifier 'id'
			print($title, field(col));
			res = (if res then res & ',' fi) & field(col);
			template('program: print("inside template", $title)')
		htiw
	rof;
	list_remove_duplicates(res, ',')

Internal template functions and stored templates called from inside the 'with' block use the book with id 'id'.

EDIT: I'm also going to implement "show_in_dialog()" that renders an html string.