DotNetAnalyzers / AspNetCoreAnalyzers

MIT License
62 stars 4 forks source link

Fix for missing parameter #52

Open JohanLarsson opened 5 years ago

JohanLarsson commented 5 years ago

Before:

namespace AspBox
{
    using System.Threading.Tasks;
    using Microsoft.AspNetCore.Mvc;

    [ApiController]
    public class OrdersController : Controller
    {
        [HttpGet(""api/orders/{↓id}"")]
        public async Task<IActionResult> GetOrder()
        {
        }
    }
}

After:

namespace AspBox
{
    using System.Threading.Tasks;
    using Microsoft.AspNetCore.Mvc;

    [ApiController]
    public class OrdersController : Controller
    {
        [HttpGet(""api/orders/{id}"")]
        public async Task<IActionResult> GetOrder(string id)
        {
        }
    }
}

Assume string if we can't glean type info from constraints.

JohanLarsson commented 5 years ago

Offer fixes for many types when unknown type