Add BindXML AND ShouldBindXML #1484

[syssam]

Add BindXML AND ShouldBindXML #1484

[codecov]

Codecov Report

Merging #1485 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1485      +/-   ##
==========================================
+ Coverage   99.15%   99.15%   +<.01%     
==========================================
  Files          36       36              
  Lines        1890     1894       +4     
==========================================
+ Hits         1874     1878       +4     
  Misses         13       13              
  Partials        3        3

Impacted Files	Coverage Δ
context.go	99.48% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bef6c56...ad32864. Read the comment docs.

[thinkerou]

Hi @syssam can you add one demo example using ShouldBindXML/BindXML? thanks!

[appleboy]

    if err := c.ShouldBindXML(&xml); err != nil {
      c.JSON(http.StatusBadRequest, gin.H{"error": err.Error()})
      return
    }
    
    if xml.User != "manu" || xml.Password != "123" {
      c.JSON(http.StatusUnauthorized, gin.H{"status": "unauthorized"})
      return
    } 
    
    c.JSON(http.StatusOK, gin.H{"status": "you are logged in"})

more readable.

[thinkerou]

agreed with @appleboy

[syssam]

make a new pr change these case? @appleboy

[appleboy]

@syssam Why need to make a new PR? I think you can change in this PR. otherwise LGTM

[thinkerou]

@syssam you should change the code and git commit and git rebase your commit message, not new pull request. If you not know git rebase usage, you should see my issue thinkerou/thinkerou#25 (comment) (sorry, Chinese issue)

[thinkerou]

LGTM, thanks! @syssam