bug reporting & a fix
This commit is contained in:
parent
07267562bc
commit
8414ab7ab3
14 changed files with 206 additions and 15 deletions
|
|
@ -1,6 +1,7 @@
|
||||||
using System.Collections.Generic;
|
using System.Collections.Generic;
|
||||||
using System.Linq;
|
using System.Linq;
|
||||||
using System.Security.Claims;
|
using System.Security.Claims;
|
||||||
|
using Microsoft.AspNet.Authorization;
|
||||||
using Microsoft.AspNet.Http;
|
using Microsoft.AspNet.Http;
|
||||||
using Microsoft.AspNet.Mvc;
|
using Microsoft.AspNet.Mvc;
|
||||||
using Microsoft.Data.Entity;
|
using Microsoft.Data.Entity;
|
||||||
|
|
@ -11,6 +12,8 @@ namespace Yavsc.Controllers
|
||||||
{
|
{
|
||||||
[Produces("application/json")]
|
[Produces("application/json")]
|
||||||
[Route("api/blog")]
|
[Route("api/blog")]
|
||||||
|
[AllowAnonymous]
|
||||||
|
|
||||||
public class BlogApiController : Controller
|
public class BlogApiController : Controller
|
||||||
{
|
{
|
||||||
private ApplicationDbContext _context;
|
private ApplicationDbContext _context;
|
||||||
|
|
@ -24,7 +27,7 @@ namespace Yavsc.Controllers
|
||||||
[HttpGet]
|
[HttpGet]
|
||||||
public IEnumerable<BlogPost> GetBlogspot()
|
public IEnumerable<BlogPost> GetBlogspot()
|
||||||
{
|
{
|
||||||
return _context.Blogspot.Where(b=>b.Visible).OrderByDescending(b=>b.UserModified);
|
return _context.Blogspot.Where(b => b.Visible).OrderByDescending(b => b.UserModified);
|
||||||
}
|
}
|
||||||
|
|
||||||
// GET: api/BlogApi/5
|
// GET: api/BlogApi/5
|
||||||
|
|
@ -145,4 +148,4 @@ namespace Yavsc.Controllers
|
||||||
return _context.Blogspot.Count(e => e.Id == id) > 0;
|
return _context.Blogspot.Count(e => e.Id == id) > 0;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
191
src/Yavsc/ApiControllers/Survey/BugApiController.cs
Normal file
191
src/Yavsc/ApiControllers/Survey/BugApiController.cs
Normal file
|
|
@ -0,0 +1,191 @@
|
||||||
|
using System;
|
||||||
|
using System.Collections.Generic;
|
||||||
|
using System.Linq;
|
||||||
|
using System.Threading.Tasks;
|
||||||
|
using Microsoft.AspNet.Http;
|
||||||
|
using Microsoft.AspNet.Mvc;
|
||||||
|
using Microsoft.Data.Entity;
|
||||||
|
using Microsoft.Extensions.Logging;
|
||||||
|
using Yavsc.Models;
|
||||||
|
using Yavsc.Models.IT.Fixing;
|
||||||
|
|
||||||
|
namespace Yavsc.ApiControllers
|
||||||
|
{
|
||||||
|
[Produces("application/json")]
|
||||||
|
[Route("api/bug")]
|
||||||
|
public class BugApiController : Controller
|
||||||
|
{
|
||||||
|
private ApplicationDbContext _context;
|
||||||
|
ILogger _logger;
|
||||||
|
|
||||||
|
public BugApiController(ApplicationDbContext context, ILoggerFactory factory)
|
||||||
|
{
|
||||||
|
_logger = factory.CreateLogger<BugApiController>();
|
||||||
|
_context = context;
|
||||||
|
}
|
||||||
|
|
||||||
|
[HttpPost("signal")]
|
||||||
|
IActionResult Signal(BugReport report)
|
||||||
|
{
|
||||||
|
// TODO check ApiKey, check duplicate, and save report
|
||||||
|
// var obj = JsonConvert.DeserializeObject(report.ExceptionObjectJson);
|
||||||
|
_logger.LogError("bug reported : "+report.ExceptionObjectJson);
|
||||||
|
if (!ModelState.IsValid)
|
||||||
|
return HttpBadRequest();
|
||||||
|
|
||||||
|
var existent = _context.Bug.Include(b=>b.False).FirstOrDefault(
|
||||||
|
f=> f.False.ShortName == report.Component &&
|
||||||
|
f.Description == report.ExceptionObjectJson &&
|
||||||
|
f.Status != BugStatus.Rejected
|
||||||
|
);
|
||||||
|
if (existent != null)
|
||||||
|
return Ok(existent);
|
||||||
|
|
||||||
|
existent = new Bug {
|
||||||
|
Description = report.ExceptionObjectJson
|
||||||
|
};
|
||||||
|
var existentFalse = _context.Feature.FirstOrDefault
|
||||||
|
(f=>f.ShortName == report.Component);
|
||||||
|
if (existentFalse==null)
|
||||||
|
{
|
||||||
|
existentFalse = new Models.IT.Evolution.Feature {
|
||||||
|
ShortName = report.Component,
|
||||||
|
Description = $"Generated by bug report, {DateTime.Now.ToLongDateString()}"
|
||||||
|
};
|
||||||
|
|
||||||
|
_context.Feature.Add(existentFalse) ;
|
||||||
|
}
|
||||||
|
existent.FeatureId = existentFalse.Id;
|
||||||
|
existent.False = existentFalse;
|
||||||
|
_context.SaveChanges();
|
||||||
|
|
||||||
|
|
||||||
|
return CreatedAtRoute("Signal", new { id = existent.Id }, existent);
|
||||||
|
}
|
||||||
|
|
||||||
|
// GET: api/BugApi
|
||||||
|
[HttpGet]
|
||||||
|
public IEnumerable<Bug> GetBug()
|
||||||
|
{
|
||||||
|
return _context.Bug;
|
||||||
|
}
|
||||||
|
|
||||||
|
// GET: api/bug/5
|
||||||
|
[HttpGet("{id}", Name = "GetBug")]
|
||||||
|
public async Task<IActionResult> GetBug([FromRoute] long id)
|
||||||
|
{
|
||||||
|
if (!ModelState.IsValid)
|
||||||
|
{
|
||||||
|
return HttpBadRequest(ModelState);
|
||||||
|
}
|
||||||
|
|
||||||
|
Bug bug = await _context.Bug.SingleAsync(m => m.Id == id);
|
||||||
|
|
||||||
|
if (bug == null)
|
||||||
|
{
|
||||||
|
return HttpNotFound();
|
||||||
|
}
|
||||||
|
|
||||||
|
return Ok(bug);
|
||||||
|
}
|
||||||
|
|
||||||
|
// PUT: api/BugApi/5
|
||||||
|
[HttpPut("{id}")]
|
||||||
|
public async Task<IActionResult> PutBug([FromRoute] long id, [FromBody] Bug bug)
|
||||||
|
{
|
||||||
|
if (!ModelState.IsValid)
|
||||||
|
{
|
||||||
|
return HttpBadRequest(ModelState);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (id != bug.Id)
|
||||||
|
{
|
||||||
|
return HttpBadRequest();
|
||||||
|
}
|
||||||
|
|
||||||
|
_context.Entry(bug).State = EntityState.Modified;
|
||||||
|
|
||||||
|
try
|
||||||
|
{
|
||||||
|
await _context.SaveChangesAsync();
|
||||||
|
}
|
||||||
|
catch (DbUpdateConcurrencyException)
|
||||||
|
{
|
||||||
|
if (!BugExists(id))
|
||||||
|
{
|
||||||
|
return HttpNotFound();
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
throw;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return new HttpStatusCodeResult(StatusCodes.Status204NoContent);
|
||||||
|
}
|
||||||
|
|
||||||
|
// POST: api/BugApi
|
||||||
|
[HttpPost]
|
||||||
|
public async Task<IActionResult> PostBug([FromBody] Bug bug)
|
||||||
|
{
|
||||||
|
if (!ModelState.IsValid)
|
||||||
|
{
|
||||||
|
return HttpBadRequest(ModelState);
|
||||||
|
}
|
||||||
|
|
||||||
|
_context.Bug.Add(bug);
|
||||||
|
try
|
||||||
|
{
|
||||||
|
await _context.SaveChangesAsync();
|
||||||
|
}
|
||||||
|
catch (DbUpdateException)
|
||||||
|
{
|
||||||
|
if (BugExists(bug.Id))
|
||||||
|
{
|
||||||
|
return new HttpStatusCodeResult(StatusCodes.Status409Conflict);
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
throw;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return CreatedAtRoute("GetBug", new { id = bug.Id }, bug);
|
||||||
|
}
|
||||||
|
|
||||||
|
// DELETE: api/BugApi/5
|
||||||
|
[HttpDelete("{id}")]
|
||||||
|
public async Task<IActionResult> DeleteBug([FromRoute] long id)
|
||||||
|
{
|
||||||
|
if (!ModelState.IsValid)
|
||||||
|
{
|
||||||
|
return HttpBadRequest(ModelState);
|
||||||
|
}
|
||||||
|
|
||||||
|
Bug bug = await _context.Bug.SingleAsync(m => m.Id == id);
|
||||||
|
if (bug == null)
|
||||||
|
{
|
||||||
|
return HttpNotFound();
|
||||||
|
}
|
||||||
|
|
||||||
|
_context.Bug.Remove(bug);
|
||||||
|
await _context.SaveChangesAsync();
|
||||||
|
|
||||||
|
return Ok(bug);
|
||||||
|
}
|
||||||
|
|
||||||
|
protected override void Dispose(bool disposing)
|
||||||
|
{
|
||||||
|
if (disposing)
|
||||||
|
{
|
||||||
|
_context.Dispose();
|
||||||
|
}
|
||||||
|
base.Dispose(disposing);
|
||||||
|
}
|
||||||
|
|
||||||
|
private bool BugExists(long id)
|
||||||
|
{
|
||||||
|
return _context.Bug.Count(e => e.Id == id) > 0;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
@ -189,6 +189,12 @@ namespace Yavsc
|
||||||
public string Topic;
|
public string Topic;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public void JoinAsync(string roomName)
|
||||||
|
{
|
||||||
|
var info = Join(roomName);
|
||||||
|
Clients.Caller.joint(info);
|
||||||
|
}
|
||||||
|
|
||||||
public ChatRoomInfo Join(string roomName)
|
public ChatRoomInfo Join(string roomName)
|
||||||
{
|
{
|
||||||
_logger.LogInformation("a client for " + roomName);
|
_logger.LogInformation("a client for " + roomName);
|
||||||
|
|
|
||||||
|
|
@ -34,6 +34,7 @@ namespace Yavsc.Services
|
||||||
_emailSender = emailSender;
|
_emailSender = emailSender;
|
||||||
siteSettings = sitesOptions?.Value;
|
siteSettings = sitesOptions?.Value;
|
||||||
hubContext = GlobalHost.ConnectionManager.GetHubContext<ChatHub>();
|
hubContext = GlobalHost.ConnectionManager.GetHubContext<ChatHub>();
|
||||||
|
_dbContext = dbContext;
|
||||||
}
|
}
|
||||||
public async Task <MessageWithPayloadResponse> NotifyEvent<Event>
|
public async Task <MessageWithPayloadResponse> NotifyEvent<Event>
|
||||||
(IEnumerable<string> userIds, Event ev)
|
(IEnumerable<string> userIds, Event ev)
|
||||||
|
|
|
||||||
|
|
@ -1,5 +1,5 @@
|
||||||
nav { background: url(/images/it/neworleans-louisiana-bywater-1973969-o.jpg);
|
nav { background: url(/images/it/neworleans-louisiana-bywater-1973969-o.jpg);
|
||||||
background-attachment: local;
|
background-attachment: local;
|
||||||
background-size: contain;
|
background-size: cover;
|
||||||
background-repeat: no-repeat;
|
background-repeat: no-repeat;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
6
src/Yavsc/wwwroot/css/main/bootstrap.css
vendored
6
src/Yavsc/wwwroot/css/main/bootstrap.css
vendored
|
|
@ -5190,10 +5190,8 @@ select[multiple].input-group-sm>.input-group-btn>.btn {
|
||||||
|
|
||||||
.navbar-brand {
|
.navbar-brand {
|
||||||
float: left;
|
float: left;
|
||||||
height: 50px;
|
padding: 9px 9px;
|
||||||
padding: 15px 15px;
|
font-size: x-large;
|
||||||
font-size: 18px;
|
|
||||||
line-height: 20px;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
.navbar-brand:hover,
|
.navbar-brand:hover,
|
||||||
|
|
|
||||||
|
|
@ -3,7 +3,6 @@
|
||||||
.navbar-nav .dropdown>a {
|
.navbar-nav .dropdown>a {
|
||||||
background-color: rgba(0, 0, 0, .2);
|
background-color: rgba(0, 0, 0, .2);
|
||||||
border-radius: 2em;
|
border-radius: 2em;
|
||||||
vertical-align: center;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
.navbar-link:hover,
|
.navbar-link:hover,
|
||||||
|
|
@ -280,13 +279,6 @@ main.container {
|
||||||
margin-left: .3em;
|
margin-left: .3em;
|
||||||
margin-right: .3em;
|
margin-right: .3em;
|
||||||
}
|
}
|
||||||
.navbar-brand {
|
|
||||||
float: left;
|
|
||||||
height: 50px;
|
|
||||||
padding: 5px 5px;
|
|
||||||
font-size: 16px;
|
|
||||||
line-height: 18px;
|
|
||||||
}
|
|
||||||
.carousel-caption-s p {
|
.carousel-caption-s p {
|
||||||
margin: 0.2em;
|
margin: 0.2em;
|
||||||
padding: .2em;
|
padding: .2em;
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue