Isto não é exatamente um problema. É so um rabicho de código que ficou pra trás e que não foi retirado.
Veja aqui:
public class Curso
{
[Key]
public int Id { get; set; }
public string Nome_Curso { get; set; }
...
public virtual Aluno Aluno { get; set; } // Este pode ser retirado
public ICollection<AlunoCurso> AlunoCursos { get; set; }
}
Como agora um aluno pode estar matriculado em vários cursos, e um curso pode ter vários alunos, esta associação:
public virtual Aluno Aluno { get; set; }
Perdeu o sentido de estar ali, e pode ser retirada.
Após retirar, não esqueça de gerar uma nova Migration para a remoção da coluna adequadamente.
EDIT
A Action está incorreta. Se você quer ler os cursos de um aluno, basta selecionar apenas o aluno e usar as propriedades de navegação:
public ActionResult MeusCursos()
{
var aluno = db.Alunos.FirstOrDefault(a => a.Usuario == User.Identity.Name);
if (aluno != null)
return View("MeusCursos", aluno);
return View();
}
Repare que troquei @model
da View:
@model IEnumerable<MeuProjeto.Models.Aluno>
@{
Layout = "/Views/Shared/_Layout.cshtml";
}
<h2>Meus Cursos</h2>
<table class="table table-hover">
<tr>
<th>
Curso
</th>
<th>
Aprovado?
</th>
<th></th>
</tr>
@foreach (var item in Model.AlunoCursos)
{
<tr>
<td>
@Html.DisplayFor(modelItem => item.Curso.Nome_Curso)
</td>
<td>
@Html.DisplayFor(modelItem => item.Aprovado)
</td>
<td>
<div class="btn-group">
<div class="col-md-offset-2 col-md-10">
@if (item.Aprovado == false)
{
<input type="submit" value="Pendente de Aprovação" name="meusCursos" class="cursos btn btn-success" disabled="disabled" data-id="@item.Id"/>
}
else
{
<input type="submit" value="Emitir Declaração" name="meusCursos" class="cursos btn btn-default" enable="enable" />
}
</div>
</div>
</td>
</tr>
}
</table>