【Django】filter()メソッドを用いているのにも関わらず、MultipleObjectsReturnedエラーが出てしまう

実現したいこと

会員制の記事公開アプリケーションをDjangoで作成しており、マイペイージ上で自身のプロフィール情報と併せて自身が投稿した記事も表示させるページを作成しています。

前提

プロフィール情報に関するモデルはaccounts/models.pyのCustomUserモデルに、記事に関するモデルはarticle/models.pyのArticleモデルで定義しており、一対多(0以上)のリレーション関係にあります。

accounts.models.py
1from django.db import models
2from django.contrib.auth.models import AbstractUser
3from django.utils.translation import gettext_lazy as _
4
5class CustomUser(AbstractUser):
6    """拡張ユーザーモデル"""
7    username = models.CharField(
8        _("username"),
9        max_length=30,
10        help_text='Required 30 characters or fewer.',
11        unique=True,
12        error_messages={
13            'unique': _("This Username already exists."),
14        },)
15    email = models.EmailField(
16        _('email'),
17        unique=True,
18        error_messages={
19            'unique': _("A user with that email address already exists."),
20        },)
21    fb_link = models.URLField(verbose_name='Facebook Link', null=True, blank=True)
22    ig_link = models.URLField(verbose_name='Instagram Link', null=True, blank=True)
23    tw_link = models.URLField(verbose_name='Twitter Link', null=True, blank=True)
24    bg_image = models.ImageField(verbose_name='Backgroung Image', null=True, blank=True, upload_to='bgimage/')
25    icon = models.ImageField(verbose_name='Icon Image', null=True, blank=True, upload_to='icon/')
26    profession = models.CharField(verbose_name='Profession', null=True, blank=True, max_length=20)
27    introduction = models.TextField(verbose_name='Introduction', null=True, blank=True, max_length=500)
28    
29    class Meta:
30        verbose_name_plural = 'CustomUser'

ariticle.models.py
1from django.db import models
2from accounts.models import CustomUser
3from mdeditor.fields import MDTextField
4
5class Article(models.Model):
6    """記事タグ"""
7    tag_choices = (
8        ('ELECTRONICS','ELECTRONICS'),
9        ('INSTALLATION','INSTALLATION'),
10        ('SERVICES','SERVICES'),
11        ('CRAFT','CRAFT')
12    )
13
14    post_user = models.ForeignKey(CustomUser, verbose_name='Post User', on_delete=models.CASCADE,)
15    title = models.TextField(verbose_name='title', max_length=50,)
16    content = MDTextField()
17    photo = models.ImageField(verbose_name='photo', null=True, blank=True,)
18    thumbnail = models.ImageField(verbose_name='thumbnail', null=True, blank=True,)
19    tag = models.CharField(verbose_name='article tag', choices=tag_choices, max_length=30,)
20    created_at = models.DateField(verbose_name='created_at', auto_now_add=True,)

該当のソースコード

マイページの表示にはDetailViewを継承したクラスを用いました。get_querysetメソッドのオーバライドは、Articleモデルに対してアクセスユーザーでフィルターを掛けてレコードを抽出するしてテンプレートへ渡す処理を行っています。

article/views.py
1from django.views import generic
2from django.contrib.auth.mixins import LoginRequiredMixin
3from accounts.models import CustomUser
4from .models import Article
5
6class MyPageView(LoginRequiredMixin, generic.DetailView):
7    template_name = 'mypage.html'
8    model = CustomUser
9    
10    def get_queryset(self):
11        articles = CustomUser.objects.filter(article__post_user=self.request.user).order_by('-created_at')
12        return articles

テンプレートは下記の様に作成しており、Work Contentブロック内にviewsで抽出したArticleモデルのレコード表示しています。

mypage.html
1{% block contents %}
2<section>
3  <div class="container">
4    <!-- Profile Block -->
5    <div class="row">
6      <div class="col-md-4 mx-auto">
7        <div class="u-pull-half text-center">
8        {% if object.icon %}
9          <img class="img-fluid u-avatar u-box-shadow-lg rounded-circle mb-3" width="200" height="auto" src="{{ object.icon.url }}" alt="Image Description">
10        {% else %}
11          <img class="img-fluid u-avatar u-box-shadow-lg rounded-circle mb-3" width="200" height="auto" src="{% static 'img/default_icon.png' %}" alt="Image Description">
12        {% endif %}
13        </div>
14      </div>
15    </div>
16    <!-- End Profile Block -->
17
18    <!-- About -->
19    <div class="row u-content-space-bottom">
20      <div class="col-lg-6 mb-5 mb-lg-5 pl-lg-5 mx-auto">
21        {% if object.introduction %}
22        <h4 class="mb-3">About me</h4>
23        <p>{{ object.introduction }}</p>
24        {% endif%}
25      </div>
26    </div>
27    <!-- End About  -->
28  </div>
29</section>
30<!-- End About Section -->
31
32
33<!-- Portfolio -->
34<section class="u-content-space">
35  <div class="container">
36    <header class="text-center w-md-50 mx-auto mb-8">
37      <h2 class="h1">Prototyping Works</h2>
38    </header>
39
40    <!-- Work Content -->
41    <div class="js-shuffle u-portfolio row no-gutters mb-6">
42      {% for article in object.article_set.all %}
43      <figure class="col-sm-6 col-md-4 u-portfolio__item" data-groups='["its-illustration"]'>
44        <img class="u-portfolio__image" src="{{ article.thumbnail.url }}" alt="Image Description">
45        <figcaption class="u-portfolio__info">
46          <h6 class="mb-0">{{ article.title }}</h6>
47          <small class="d-block">Branding</small>
48        </figcaption>
49        <a class="js-popup-image u-portfolio__zoom" href="assets/img-temp/portfolio/img1.jpg">Zoom</a>
50      </figure>
51      {% endfor %}
52      <!-- sizer -->
53      <figure class="col-sm-6 col-md-4 u-portfolio__item shuffle_sizer"></figure>
54    </div>
55    <!-- End Work Content -->   
56  </div>
57</section>
58<!-- End Portfolio -->
59{% endblock %}

発生している問題・エラーメッセージ

python manage.py runseruverを実行するとブラウザ上で下記のエラーメッセージが表示されてしまいます。

MultipleObjectsReturned at /mypage/username/
get() returned more than one CustomUser -- it returned 2!
Request Method:	GET
Request URL:	http://127.0.0.1:8000/mypage/username/
Django Version:	3.0.3
Exception Type:	MultipleObjectsReturned
Exception Value:	
get() returned more than one CustomUser -- it returned 2!
Exception Location:	/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/db/models/query.py in get, line 422
Python Executable:	/Users/xxx/anaconda3/envs/venv_prototyping/bin/python
Python Version:	3.7.6
Python Path:	
['/Users/xxx/Desktop/prototyping',
 '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python37.zip',
 '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7',
 '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/lib-dynload',
 '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages']

Traceback (most recent call last):
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/core/handlers/base.py", line 115, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/base.py", line 71, in view
    return self.dispatch(request, *args, **kwargs)
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/contrib/auth/mixins.py", line 52, in dispatch
    return super().dispatch(request, *args, **kwargs)
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/base.py", line 97, in dispatch
    return handler(request, *args, **kwargs)
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/detail.py", line 106, in get
    self.object = self.get_object()
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/detail.py", line 52, in get_object
    obj = queryset.get()
  File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/db/models/query.py", line 422, in get
    num if not limit or num < limit else 'more than %s' % (limit - 1),

Exception Type: MultipleObjectsReturned at /mypage/username/
Exception Value: get() returned more than one CustomUser -- it returned 2!

試したこと・調べたこと

Djangoの管理画面から試しに当該ユーザに紐づけられるArticleモデルのレコードを2つではなく1つで実行してみたところ、正常に表示されました。

このエラーがどういう状況で発生するか下記ページで調べてみたところ、get()メソッドを使用した際にレコードが2つ以上ある場合に発生するエラーで、get()メソッドをfilter()メソッドにすることで解決することが分かりました。
https://torajirousan.hatenadiary.jp/entry/2019/02/08/230027

しかし今回はget()メソッドを使わずにfilter()メソッドを使用しており、なぜこのようなエラーが出てしまうのか分かりませんでした。

補足情報（FW/ツールのバージョンなど）

開発環境は下記の通りです
・python 3.7.6
・Anacondaローカル内
・Django3系
・MacOS
・Chromeブラウザ

回答1件

ベストアンサー

get_queryset()が問題ではなく、DetailViewが使用しているget_object()メソッドが原因です。
DetailViewは、デフォルトでget_object()メソッドを使用して、クエリセットから単一のオブジェクトを取得しようとします。
get_object()メソッドは内部で get()を使用しており、ここで複数のオブジェクトが返されると MultipleObjectsReturnedエラーが発生します。
MyPageViewクラスでget_queryset()をオーバーライドしても、get_object()メソッドはオーバーライドされていないため、デフォルトの実装が使用されます。
ですのでget()メソッドが使用され、エラーが発生しています。

投稿2023/04/30 10:12

NICKO

総合スコア62

回答へのコメント

Mr_PONPON_MARU

2023/04/30 11:16

コメントありがとうございます。DetailViewでget_object()メソッドがデフォルトで働いていることは知りませんでした。修正案として、MyPageViewをListViewを継承したクラスビューへと変更するか、get_queryset()メソッドを削除して、下記のようにget_object()メソッドを修正する必要がある、という認識でよろしいでしょうか？ ```views.py class MyPageView(LoginRequiredMixin, generic.DetailView): template_name = 'mypage.html' model = CustomUser def get_object(self): articles = CustomUser.objects.filter(article__post_user=self.request.user).order_by('-created_at') return articles ```

NICKO

2023/04/30 11:21

その認識であっています。ただしDetailViewを使用する場合は、単一のオブジェクトを返すことが期待されていますので、記事のリストを返すためには、代わりにListViewを使用することをオススメします。

Mr_PONPON_MARU

2023/04/30 12:43 編集

ありがとうございます。もう一点素人質問で恐縮ですが、従テーブルのArticleからは複数オブジェクトを、主テーブルのCustomUserからは単一のオブジェクトを返す今回の事例のように、常に主テーブルのデータは単一で返され、主テーブルから複数のデータを返すことが想定されない場合でもListViewを継承したクラスビューを用いる事は、ビュー設計の観点的には問題はありませんでしょうか？

NICKO

2023/04/30 22:02

返信遅れて申し訳ございません。はい。特に問題はございません。

Mr_PONPON_MARU

2023/05/01 05:04

ListViewに変えて試してみます。ありがとうございました。