EPrints Technical Mailing List Archive

Message: #08562


< Previous (by date) | Next (by date) > | < Previous (in thread) | Next (in thread) > | Messages - Most Recent First | Threads - Most Recent First

Re: [EP-tech] View and Move Screen Plugins


CAUTION: This e-mail originated outside the University of Southampton.

Hi!

 Thanks for the suggestions. Using 101 or 102 does not change. I've used grep to find "EPrint::Move" or "Move to Repository", just found the standard ones.

I'm looking to every plugin installed but everything is ok.

So, what is the point where it create the form code, and maybe its caller? Maybe I can debug there and see what happen.

Il 01/04/21 12:13, David R Newman ha scritto:

Hi Yuri,

This is an odd one, as if the "Move to the Repository" button does not actually do what it says when the setting is the default, position = 100, then it seems extremely unlikely that this would not have been identified as a bug by now, as this would likely apply to all repositories running this version.

It is possible to reconfigure these values in the local archive's configuration, (typically plugins.pl).  It is worth checking there to see if there is anything that relates to the EPrints::Move screen plugin.  It may also be worth checking you archive's phrase files to see if the is a "Move to the repository" assigned to another button other than the one from EPrints::Move.  Beyond, that all I can suggest is try setting the position to 101 rather than 200 to see if this fixes the functionality of the button without changing the ordering of buttons. 

This sounds like one of the problems I would have difficultly debugging even if the code was in front of me.  So I am not overly confident any one of my suggestions will fix your problem.  If this problem has only just occurred, I would be tempted to run a find command over all your archive config and core EPrints codebase files, to see if anything relevant has recently been modified.

Regards

David Newman

On 01/04/2021 10:50, Yuri via Eprints-tech wrote:
CAUTION: This e-mail originated outside the University of Southampton.

Context: Eprints 3.3.15

When I view an Eprints (as admin) I get the "Move to the repository", "Return Item (with notification)", "Delete Item (with notification)" buttons on top (eprint_actions_bar_buffer).

The standard config for "Move to the repository" ( perl_lib/EPrints/Plugin/Screen/EPrint/Move.pm ) is

{ place => "eprint_actions_bar_buffer", action => "move_archive", position => 100, },

The form render as:

<form method="post" accept-charset="utf-8" action="" enctype="multipart/form-data">
    <input name="screen" id="screen" value="EPrint::View" type="hidden">
    <input name="eprintid" id="eprintid" value="263" type="hidden">
    <input class="ep_form_action_button" value="Move to Repository" name="_action_move_archive" type="submit">
</form>

As you can see, the form points to EPrint::View instead of EPrint::Move.

If I change the config to:

{ place => "eprint_actions_bar_buffer", action => "move_archive", position => 200, },

It displays correctly EPrint::Move but "Return Item (with notification)" move to the first position and have the same issue, it renders with EPrint::View instead of EPrint::Move.

So the first button always get EPrint::View, the others EPrint::Move. Obviously the one with EPrint::View fails with "This screen (EPrint::View) does not know how to process this action: move_archive".

Any idea on how to solve this?

P.S: This is the full config for Move.pm (the standard one):

    $self->{actions} = [qw/ move_inbox move_buffer move_archive move_deletion /];

    $self->{appears} = [
{ place => "eprint_actions",     action ="" "move_inbox",     position => 600, },
{ place => "eprint_editor_actions",     action ="" "move_archive",     position => 400, },
{ place => "eprint_editor_actions",     action ="" "move_buffer",     position => 500, },
{ place => "eprint_editor_actions",     action ="" "move_deletion",     position => 700, },
{ place => "eprint_actions_bar_buffer", action ="" "move_archive", position => 100, },
{ place => "eprint_actions_bar_archive", action ="" "move_buffer", position => 100, },
{ place => "eprint_actions_bar_archive", action ="" "move_deletion", position => 100, },
{ place => "eprint_actions_bar_deletion", action ="" "move_archive", position => 100, },
{ place => "eprint_review_actions", action ="" "move_archive", postion => 200, }, # I've already correct postion -> position in the instance
    ];

*** Options: http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
*** Archive: http://www.eprints.org/tech.php/
*** EPrints community wiki: http://wiki.eprints.org/

Virus-free. www.avg.com
-- 
Yuri Carrer

 CAB - Centro di Ateneo per le Biblioteche, Università di Padova
 Tel: 049/827 9712 - Via Beato Pellegrino, 28 - Padova