am-impact / amnav

Navigation plugin for Craft
168 stars 19 forks source link

Bugfix for assuming there's always an active node #29

Closed boboldehampsink closed 9 years ago

boboldehampsink commented 9 years ago

Bugfix for assuming there's always an active node

hubertprein commented 9 years ago

You're correct, I missed that one!

You're missing a semicolon on line 359 though. Not merging just yet! :)

boboldehampsink commented 9 years ago

That was sloppy! Fixed now

boboldehampsink commented 9 years ago

For a nicer commit I will squash both commits, hang on.

boboldehampsink commented 9 years ago

Ready for merge

hubertprein commented 9 years ago

Thanks Bob!

boboldehampsink commented 9 years ago

Could you release this as hotfix? I have multiple projects that run into errors and I want to update this via composer.

hubertprein commented 9 years ago

Done.