Cleanup and fixes for some cases #21
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this PR:
usesclause of most of the units that are not needed, and they make it for a bloated and in cases (with C++Builder specially) not working units for the client code.StampRequestwas storing the passed CFDI into a "temporary" file, saved to the current directory. This won't work in lots of cases, for instance, if the user doesn't have privileges to modify the current folder (which will be the case for a lot of users as the application folders should not be modified in general without privileges), or there are several concurrent users that need to use this services at the same time. The fix was to just pass the XML in memory, instead of storing it into disk and then reading it again.uuidfield in the response because that property is a dynamic key (the UUID is the key) so the RTTI can't automatically parse this. This has been reported to SmarterWeb with case MDA-68926