Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Phingcall should have the options returnProperty (Trac #1209) #576

Closed
phing-issues-importer opened this issue Oct 9, 2016 · 5 comments
Closed

Comments

@phing-issues-importer
Copy link

phing-issues-importer commented Oct 9, 2016

Currently it's not possible to return values from phing targets (due to them being executing in a new scope) which is difficult to work with.

I propose a propery returnProperty for the phingcall task and something akin to in Tasks.

Migrated from https://www.phing.info/trac/ticket/1209

{
    "status": "new", 
    "changetime": "2016-10-07T08:28:55", 
    "description": "Currently it's not possible to return values from phing targets (due to them being executing in a new scope) which is difficult to work with.\n\nI propose a propery returnProperty for the phingcall task and something akin to <return value=\"myReturnValue\" /> in Tasks.", 
    "reporter": "hoferlukas1@gmail.com", 
    "cc": "", 
    "resolution": "", 
    "_ts": "1475828935569836", 
    "component": "phing-core", 
    "summary": "Phingcall should have the options returnProperty", 
    "priority": "tbd", 
    "keywords": "phingcall,return,returnProperty", 
    "version": "2.11.0", 
    "time": "2015-08-17T12:19:06", 
    "milestone": "4.0", 
    "owner": "mrook", 
    "type": "enhancement"
}
@phing-issues-importer
Copy link
Author

phing-issues-importer commented Oct 9, 2016

Trac update at 20151204T10:13:48: mrook commented:

As you said, phingcall executes in a completely new scope, which is dumped after the execution finishes. Ideas are welcome :)

@phing-issues-importer
Copy link
Author

phing-issues-importer commented Oct 9, 2016

Trac update at 20160804T21:29:57:

  • mrook commented:

I'm putting this as an enhancement for 3.0, as it's a non-trivial change.

  • mrook changed milestone from "Backlog" to "3.0"

@phing-issues-importer
Copy link
Author

phing-issues-importer commented Oct 9, 2016

Trac update at 20161007T08:28:55:

  • mrook commented:

Milestone renamed

  • mrook changed milestone from "3.0" to "4.0"

@mrook mrook removed their assignment Nov 17, 2016
@mrook
Copy link
Member

mrook commented Dec 29, 2016

The runtarget task in #527 could be an option.

@mrook
Copy link
Member

mrook commented Jan 28, 2017

Closing because #527 was merged.

@mrook mrook closed this as completed Jan 28, 2017
@mrook mrook modified the milestones: 3.0, 4.0 Jan 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants