- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 29
Add tests for local elements when elementFormDefault is set #46
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
base: master
Are you sure you want to change the base?
Conversation
806f479    to
    c78b78d      
    Compare
  
    | The test is fixed with both goetas-webservices/xsd2php#108 and goetas-webservices/xsd-reader#52. I have used https://github.com/gossi/composer-localdev-plugin to symlink to local development packages, works like a charm. Please wait before merging those pull request, as I wish to add more test cases based on this ID3 Webservice, and try integration in my real world app. | 
c78b78d    to
    f277e28      
    Compare
  
    | Hi @goetas I pretty sure you are really busy on many other subjects, but you could review and merge this pull requests and others related to this one ? 
 I'm currently running in an app that's now live in production the following patches from my pull requests. Everything is working properly with those patches, i'm pretty sure you could merge those pull requests and perform new release with confidence :) Please let me know, best regards. | 
| Hi, sorry if this is taking so long, but recently i've been really busy. I'm really interested in goetas-webservices/xsd-reader#52 (and in the scenario that fixes) but I would like to have time to test it a bit better (I tend to follow the SOAP interpretation done by SoapUI... so would like to have some time to test it with it). I think that the quality of both pull requests is very good, is just that i would like to find time to test them properly. | 
| same note for the tests as in goetas-webservices/xsd2php#108 | 
7ca2a9d    to
    e31a4d4      
    Compare
  
    | Test have been simplified. They fail on CI because they need to run with goetas-webservices/xsd2php#108 and goetas-webservices/xsd-reader#52 | 
| Shouldnt we first merge goetas-webservices/xsd2php#108? To make tests pass you can add  | 
e31a4d4    to
    aadcbde      
    Compare
  
    aadcbde    to
    8fa3f8c      
    Compare
  
    | Thanks ! Thanks. I think we also need to release wsdl2php with this change in composer.json : goetas-webservices/wsdl2php#12 | 
Related to goetas-webservices/xsd-reader#51
This currently adds a test in soap-client matching the issue. (and test is actually failing to demonstrate the issue)