Enhancement (DELETE) and Bug fixes (around POST response)

1. In fetch(), the check for negative response to raise HALNavigatorError,
       if raise_exc and not self.response:
            raise HALNavigatorError(self.response.text,
                                    status=self.status,
                                    nav=self,
                                    response=self.response,
            )    
   Is not too late in the flow?. I feel it should be checked before processing the response body 

2. PostResponse can have properties, hence implemented __call__ to return properties

3. DELETE method is similar to POST from Navigator perspective (non-idempotent) 
  Hence,
         a) Reused logic from POST to implement support for DELETE
         b) renamed PostResponse to HALResponse 
         c) renamed  create() to post() for clarity
   In order to provide backward compatibility, I created alias "create"  to "post"

4. HALNavigator should not allow POST if the uri is templated. The check for templated_uri 
   was missing. Created a wrapper function (@template_uri_check) to re-use for all methods - GET/POST/DELETE

TODO:
1. Enforce templated_uri check for all HTTP calls. Add test coverage 
2. a link usually have 'method' property (default to 'GET'), which can be used to restrict http operations. this helps to raise NotImplementedError and can have a controlled default function e.g autofetch
Makes Sense!
On 31 Oct 2014 08:28, "Josh Kuhn" <notifications@github.com> wrote:

> In restnavigator/halnav.py:
>
> > @@ -369,12 +461,13 @@ def docsfor(self, rel):
> >          print('opening', doc_url)
> >          webbrowser.open(doc_url)
> >
> > -class PostResponse(HALNavigator):
> > -    '''A Special Navigator that is the result of a POST
> > +
> > +class HALResponse(HALNavigator):
>
> I am thinking of naming this class something like "OrphanResource".
> Basically, this is good for post/delete/ etc that don't have a URL, but it
> will also be good for embedded documents that don't have a self link (which
> the HAL spec allows), but which we'll want to treat like a Navigator in all
> respects except being able to refetch it
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/deontologician/rest_navigator/pull/8/files#r19656040>.
>