Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 8, 2020. It is now read-only.

added zendframework/zendxml into suggest at Zend\Json's composer.json#6778

Merged
Ocramius merged 5 commits into
zendframework:masterfrom
samsonasik:zendxml-composer-json
Nov 14, 2014
Merged

added zendframework/zendxml into suggest at Zend\Json's composer.json#6778
Ocramius merged 5 commits into
zendframework:masterfrom
samsonasik:zendxml-composer-json

Conversation

@samsonasik
Copy link
Copy Markdown
Contributor

@samsonasik samsonasik commented Oct 18, 2014

used by Zend\Json\Json::fromXml()

@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Oct 19, 2014

Should probably be optional. Additionally, I'd suggest just deprecating this API, as it's just a problem to couple the Json component to zendframework/zendxml

@Ocramius Ocramius self-assigned this Oct 19, 2014
@samsonasik samsonasik changed the title added zendframework/zendxml into require at Zend\Json's composer.json added zendframework/zendxml into suggest at Zend\Json's composer.json Oct 19, 2014
@samsonasik
Copy link
Copy Markdown
Contributor Author

samsonasik commented Oct 19, 2014

@Ocramius done ;). I've moved the zendxml to suggest and mark Json::fromXml() as @deprecated .

@adamlundrigan
Copy link
Copy Markdown
Contributor

adamlundrigan commented Nov 14, 2014

👍. I would prefer to see a separate "XmlToJson" component that composes both Zend\Json and Zend\Xml to perform the conversion, rather than direct dependency between the two components.

My only suggestion here is to note why the method is deprecated - either a short blurb or a link to this ticket - so that when it comes time to update this component for ZF3 we'll know why it was done.

@samsonasik
Copy link
Copy Markdown
Contributor Author

samsonasik commented Nov 14, 2014

@adamlundrigan I have added this PR id after @deprecated docblock

@gianarb
Copy link
Copy Markdown
Contributor

gianarb commented Nov 14, 2014

@samsonasik 👍

@Ocramius Ocramius added this to the 2.3.4 milestone Nov 14, 2014
@Ocramius Ocramius merged commit a08972e into zendframework:master Nov 14, 2014
Ocramius added a commit that referenced this pull request Nov 14, 2014
@Ocramius
Merge branch 'hotfix/#6778-zendxml-suggestion-in-zend-json'
eaf9dd3 
Close #6778
Ocramius added a commit that referenced this pull request Nov 14, 2014
@Ocramius
Merge branch 'hotfix/#6778-zendxml-suggestion-in-zend-json' into develop
d06f74e 
Close #6778
Forward port #6778
@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Nov 14, 2014

@samsonasik merged, thx!

master: eaf9dd3
develop: d06f74e

@samsonasik samsonasik deleted the zendxml-composer-json branch November 14, 2014 23:53
@samsonasik samsonasik mentioned this pull request Nov 18, 2014
Closed
Ocramius added a commit that referenced this pull request Nov 18, 2014
@Ocramius
Merge branch 'hotfix/#6891-#6778-composer-json-string-escaping'
88c124c 
Close #6891
Ocramius added a commit that referenced this pull request Nov 18, 2014
@Ocramius
Merge branch 'hotfix/#6891-#6778-composer-json-string-escaping' into …
2851a22 
…develop

Close #6891
Forward port #6891
gianarb pushed a commit to zendframework/zend-json that referenced this pull request May 15, 2015
@Ocramius
Merge branch 'hotfix/zendframework/zendframework#6778-zendxml-suggest…
cb7491a 
…ion-in-zend-json'

Close zendframework/zendframework#6778
gianarb pushed a commit to zendframework/zend-json that referenced this pull request May 15, 2015
@Ocramius
Merge branch 'hotfix/zendframework/zendframework#6778-zendxml-suggest…
3cbb921 
…ion-in-zend-json' into develop

Close zendframework/zendframework#6778
Forward port zendframework/zendframework#6778
@rakesh-beedasy
Copy link
Copy Markdown

rakesh-beedasy commented May 4, 2016

Why is Zend\Xml2Json\Xml2Json::fromXml marked as deprecated in zf3 ?

@weierophinney
Copy link
Copy Markdown
Member

weierophinney commented May 4, 2016

Why is Zend\Xml2Json\Xml2Json::fromXml marked as deprecated in zf3 ?

It's not; the method for doing so in Zend\Json\Json has been extracted into its own package, zendframework/zend-xml2json.

@rakesh-beedasy
Copy link
Copy Markdown

rakesh-beedasy commented May 4, 2016

https://github.com/zendframework/zend-xml2json/blob/master/src/Xml2Json.php#L45

samsonasik added a commit to samsonasik/zend-xml2json that referenced this pull request May 4, 2016
@samsonasik
remove @deprecated note comes from zendframework/zendframework/pull/6778
005c02e 


in favor of zendframework/zendframework#6778 (comment)
@samsonasik samsonasik mentioned this pull request May 4, 2016
Merged
@samsonasik
Copy link
Copy Markdown
Contributor Author

samsonasik commented May 4, 2016

@rakesh-beedasy I've created PR for @deprecated doc removal for that: zendframework/zend-xml2json#1

@rakesh-beedasy
Copy link
Copy Markdown

rakesh-beedasy commented May 4, 2016

Thanks 👍

weierophinney added a commit to zendframework/zend-xml2json that referenced this pull request Oct 16, 2017
@weierophinney
Merge pull request #1 from samsonasik/patch-1
100f6e4 
remove @deprecated doc came from zendframework/zendframework/pull/6778
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@Ocramius Ocramius

Labels

bug Json

Projects

None yet

Milestone

2.3.4

Development

Successfully merging this pull request may close these issues.

6 participants

@samsonasik @Ocramius @adamlundrigan @gianarb @rakesh-beedasy @weierophinney