give access to $__user_email__ and $__user_id__ variables in hidden tool parameters.

#206 Declined
Repository
Deleted repository
Branch
abretaud/__init__py-edited-online-with-bitbucket-1377096444205 (46291c572e14)
Repository
galaxy-central
Branch
default
Author
  1. Anthony Bretaudeau
Reviewers
Description

This patch allow to use $user_email and $user_id variables in hidden tool parameters. This is useful for example when adding to galaxy a data source that needs to identify the user who is willing to import data in a galaxy instance. Here is an example:

     <inputs action="https://example.com/my/data/source"
check_values="false" method="get">
         <display>Browse the XX data source $GALAXY_URL</display>
         <param name="user_email" type="hidden" value="$__user_email__" />
     </inputs> 

There are 2 messages in the galaxy-dev mailing list talking about that: http://dev.list.galaxyproject.org/Reserved-variables-in-param-tags-tt4656402.html http://dev.list.galaxyproject.org/Passing-user-email-to-a-data-source-tt4661306.html

Comments (4)

  1. John Chilton

    I would like the LWR job runner to have expand certain job parameters the same way so I am interested in this functionality. I have expanded the approach here in such a way that it is centralizes some similar functionality found else where (dynamic options & job templates).

    https://github.com/jmchilton/galaxy-central/commit/662dcf039d7a67b2a293a4e3453ec5dcc0238fc0 https://github.com/jmchilton/galaxy-central/commit/662dcf039d7a67b2a293a4e3453ec5dcc0238fc0.diff

    I need to do some more testing before issuing a pull request or I might merge in into the LWR pull request. Any objections to it or comments?

    Thanks!

  2. Anthony Bretaudeau author

    Hello, I don't have the time to test your patch right now, but if it works, no problem to replace my pull request with it! Thanks!

  3. John Chilton

    I have test this functionality and it seems to work with pull request 224. I would recommend merging that request.

  4. John Chilton

    I have pushed a changeset with equivalent functionality to galaxy-central directly. Thanks for the contribution!