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.

Sign up for GitHub

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

Jump to bottom

Enable concatenation of flows #102

Closed
mgeipel opened this issue Jun 17, 2013 · 0 comments
Closed

Enable concatenation of flows #102

mgeipel opened this issue Jun 17, 2013 · 0 comments
Assignees
@mgeipel

Comments

@mgeipel
Copy link

mgeipel commented Jun 17, 2013

@NAME is used as a place holder. This allows for joining of several flows. The pattern can be used to implement an equivalent of an SQL join.

"A"|
template("${o}")|
template("x${o}")|
@X;

"B"|
template("x${o}")|
@X;

@X|
template("${o}")|
write("stdout");
@ghost ghost assigned mgeipel Jun 17, 2013
@mgeipel mgeipel mentioned this issue Jun 17, 2013
Closed
@fsteeg fsteeg mentioned this issue Jun 17, 2013
Closed
cboehme pushed a commit to cboehme/metafacture-core that referenced this issue Jul 25, 2017
fixed metafacture#102
00146ea
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Add updated tests from 65e1de9 (see #95, #102)
248d18d
blackwinter added a commit that referenced this issue Dec 13, 2024
@fsteeg @blackwinter
Fix tests based on feedback on 248d18d (see #95, #102)
ca2ffb8 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Extract find* and transform* methods from Value.Array (#102)
4be871b 
To a `Path` class that in the end should become something like
https://metacpan.org/pod/Catmandu::Path::simple
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Extract find* and transform* methods from Value.Hash (#102)
5cbe20a
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Rename Path to FixPath as suggested in feedback on 2a14c70 (#102)
821308f
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Extract insert* and removeNested logic from Value (#102)
1b96789
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Order FixPath members by visibility, unify naming (#102)
ef84bc8
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Restrict visibility and improve naming consistency (#102)
f85cc3e
blackwinter added a commit that referenced this issue Dec 13, 2024
@fsteeg @blackwinter
Integrate feedback from #133 (see #102)
a9aac45 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Extract remaining append logic from Value (#102)
ca5c9eb
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Add disabled test for unimplemented nested array case (#102)
8decd3f
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Remove findNested, align structure of transformIn / findIn (#102)
45d0182
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Merge branch '102-path' (#102)
6b05dcf 
See metafacture/metafacture-fix@45d0182
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Unify ReservedField usage, align structure of insert methods (#102)
425dc49 
Causes minor behavior changes (see new @MetafixToDo annotations)

Contains comments sketching possible ways to consolidate, basically
on two levels: 1) implement transform/insert/remove in terms of find
(find does all path traversal, then the result is changed / deleted)
2) move hash / array methods for each operation into hash / array

Short-term next step I'd look into: remove / unify InsertMode usage
(hence the somewhat odd null / default InsertMode in `insertInto`)
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Replace existing values in set_array and set_hash (#102, #111)
1a3278a
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Remove appendTo method, hide FixPath behind record (#102, #92, #89)
8c42874 
Replace replace*/append* methods with insert* and InsertMode usage

Use get / set / add / remove methods on Record for interaction
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Implement transformIn using findIn & insertInto (#102, #121, #147)
c25ede3 
For cases where the findIn path does not work with insertInto
(like `title-?` or `some.*.nested.*`), use a new Value#path field
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Move transform methods to Record, avoid FixPath in FixMethod (#102)
692b4f9
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Update passing integration tests (#102, #92, #121, #149, #155)
61e3748
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Don't use path in equals, hashCode and toString methods (#102)
34c96f1
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Tweak lookup behavior, partial delete support (#102, #149)
e9dd4ab
blackwinter added a commit that referenced this issue Dec 13, 2024
@fsteeg @blackwinter
Integrate feedback from #170 (see #102)
1bcc5ca 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Throw exception on transform of non-string field (#102, #100, #133)
be764e4
blackwinter added a commit that referenced this issue Dec 13, 2024
@fsteeg @blackwinter
Integrate feedback from #170 (see #102)
8b27a52 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter added a commit that referenced this issue Dec 13, 2024
@fsteeg @blackwinter
Some cleanup as suggested in #170 (see #102)
93aaf2e 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter added a commit that referenced this issue Dec 13, 2024
@fsteeg @blackwinter
Handle null results directly in transform (#102, #149, #170)
238e840 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Remove unnecessary parens, add unit test (#102, #149, #170)
006e3ac
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Add test cases from review (#102, #170)
5494f42 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>

See ad0f3d5, 33d103f, 771205f, 49849a4, fa24c36, d5e67fc
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Remove untested $append, $first, $last support on hashes (#102)
1e8ac6e
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
First lookup error #102
daf0394
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Second lookup error #102
4f82d96
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Add new replace_all error (#102)
aa585b0
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Add new replace_all error (#102)
80ad4ef
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Third lookup error #102
4055a43
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Split+Trim+Move error #102
c1003b3
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Copy_field with index error #102
872d0f2
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Conform tests to editorconfig #102
55f2ba8
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Add test variant and fix tests #102
d199059
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Update passing integration tests (#102, #149, #170)
e027e1c
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Add integration test for replace with last Wildcard (#102)
389826b
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Add integration test for add_field with $last wildcard
25c2c7b 
#102

Behaviour seems to have changed #102 breaks and master ingores
if there is no object in the array and you use to `add_field` with
$last wildcard
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Merge pull request #204 from metafacture/102-oersiIntegration
6c5403c 
Add integration test for replace with last Wildcard (#102)
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Only create new path if old value's path doesn't match (#102, #170)
64d8d98
blackwinter added a commit that referenced this issue Dec 13, 2024
@blackwinter
Add unit test for record transformation. (#102, #170)
8cfe1a8
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Fix nested removal from array (#102, #113, #170)
e2cf74c
blackwinter added a commit that referenced this issue Dec 13, 2024
@blackwinter
Add unit test for record transformation. (#102, #170)
2378455
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Preserve order in Record#transform (#102, #170)
f210dd3 
Mark tests covering faulty usage as TODOs
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Explicitly set path in record test (#102, #170)
a4d557e
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Delete reversed null results after ordered transform (#102, #170)
ec9f8ef
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Mark failing strictness tests as TODOs (#102, #170)
40ab054
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Update path for repeated fields to indexed array (#102, #170)
4324378
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Update path when adding, rename on copy/move field (#102, #170)
ca1cba1
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Set correct path after inserting transformed value (#102, #170)
049c709
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Replace matchType.ifString.orElseThrow with asString (#102, #170)
31cf8e9
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@fsteeg
Improve * wildcard behavior, additional tests (#102, #170)
6779544
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mgeipel mgeipel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mgeipel