docs/examples use different resource-notation about raisin HOT 5 CLOSED

Could you show full example, please.

Because both of them right. It depends how you call it.

For instance:

resource subres => sub { ... }

and

desc 'subres desc',
resource => 'subres' => sub { ... }

from raisin.

I have 1 pm mounting 4 other pm's. I implemented only your below example, because when I use the top one then I get the afore-mentioned errors.

# /person
desc 'person-related methods',
resource => person => sub { .. 

    # /person/<id>
    route_param id => Int,
    sub {
        # get person 
        desc "Show a person's data",
    ...

    };
};

I didn't realize both were possible; I'll check the docs again ;-) Thanks.

from raisin.

Maybe you could give me an advice?

Is current syntax clear for you?
Maybe it would be better if I changed the syntax, where all keywords would be act as a separate subroutine?

For instance:

desc 'some action';
params required => { name => 'somename', type => Int }, optional => { name => 'optname', type => Str };
get sub {
  ...
};

Or it will be better to leave as it is?

from raisin.

Thanks for your feedback and checking with me.

My take on this is that unless there's a clear reason (and description) why there are 2 different notations (and when & how to use them), it might indeed be better to use only 1 consistent notation that does not allow for any guesswork.

Also, at least in my experience, too many different ways of allowing definitions and/or too much aliasing to remain backwards compatible almost invariably end up being a maintenance and testing nightmare.

You'll have to decide eventually (because there is always a risk that by refactoring this, existing (albeit only a few weeks' or months' old) code could break), but I for one will be sticking with only 1 notation, like my earlier example, as it makes the raisin parser behave as I would expect.

Thanks again for the great work on Raisin. Very much appreciated!

Best,
Chris.

from raisin.

You can use Raisin::API '-old' for your old API.

from raisin.