Skip to content
Snippets Groups Projects

Wrap in array if needed

Merged Wrap in array if needed
TREFILEK/rpc-netid-php:master into master
1 unresolved thread
Merged Matt Trefilek requested to merge TREFILEK/rpc-netid-php:master into master
1 unresolved thread

This seems very hacky, so I am okay finding a different solution, but it maintains the structure and passes all tests.

@bjsousa

Merge request reports

Approval is optional

Merged by avatar (Apr 11, 2025 4:09pm UTC)

Merge details

  • Changes merged into master with b299ce8b.
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
src/main/edu/wisc/doit/RpcNetidStructValidationResponse.php
25 25 * @param array $reasons the array of a Reason for why the SOAP Call failed
26 26 */
27 27 function __construct( $isValid, array $reasons ) {
28 $this->setIsValid( $isValid );
28 if(gettype($reasons) == "array") {
29 $this->setIsValid( $isValid );
30 } else {
31 $this->setIsValid(array( $isValid) );
  • Benjamin Sousa
    Benjamin Sousa @bjsousa ·
    Owner

    Let's try it :thumbsup:

  • Matt Trefilek merged

    merged

  • Matt Trefilek mentioned in commit b299ce8b

    mentioned in commit b299ce8b

    • Please register or sign in to reply