CODEWITHSUNDEEP
pythondjango
AlexW
AlexW

Reputation: 2589

Django template - get matching foreign key objects in a template?

I have a model named circuits and each circuit may have many files associated to it.

so in my view i am getting all the circuits and all the files, then i am trying to get the files to match the circuits in my template using an if statement.

i have tried, the below but all come back empty, how do i traverse models in a template to find a match?

file.circuit_contract_data
file.circuit_contract_data__id

Thanks

view.py

@login_required    
def showroom_detail(request, showroom_id):
    modelShowroom = get_object_or_404(ShowroomConfigData, pk=showroom_id)
    modelCircuits = CircuitInfoData.objects.filter(showroom_config_data=showroom_id) 
    modelCircuitFiles = CircuitFiles.objects.filter(circuit_contract_data__showroom_config_data=showroom_id) 
    modelSitePhotos = SitePhotos.objects.filter(showroom_config_data=showroom_id)
    modelSiteFiles = SiteFiles.objects.filter(showroom_config_data=showroom_id)    

    return render(request, 'service/showroom_detail.html', {
        'Showroom': modelShowroom,
        'Circuits': modelCircuits,
        'CircuitFiles': modelCircuitFiles,
        'SitePhotos': modelSitePhotos,
        'SiteFiles': modelSiteFiles,
    })

template

{% for item in Circuits %}
    <tr class="{% cycle 'tr-1' 'tr-2' %}">  
        <td>{{ item.provider }}</td>
        <td>{{ item.service_type }}</td>
        <td>{{ item.circuit_speed }}</td>
        <td>
        {% for file in CircuitFiles %}
            {% if file.circuit_contract_data == item.id %}
            <a href ="{{ MEDIA_URL}}{{ file.circuit_file }}" target="_blank">{{ file.file_name }}</a><br />
            {% endif %} 
        {% endfor %}
        </td>
    </tr>

model

class CircuitInfoData(models.Model):    
    showroom_config_data = models.ForeignKey(ShowroomConfigData,verbose_name="Install Showroom",blank=True)
    major_site_info = models.ForeignKey(MajorSiteInfoData,verbose_name="Install Site",blank=True)        
    service_type = models.ForeignKey(CircuitTypeInfoData)
    circuit_speed = models.IntegerField(blank=True)
    circuit_bearer = models.IntegerField(blank=True)
    provider = models.CharField(max_length=200)
    ref_no = models.CharField(max_length=200,verbose_name="Reference No")
    install_location = models.CharField(max_length=200)
    install_date = models.DateField()
    cost_per_month = models.DecimalField(decimal_places=2,max_digits=8)
    contract_length = models.IntegerField(verbose_name="Contact length in years")
    notes = models.TextField(blank=True)
    service_service_contacts = models.ForeignKey(ServiceContacts)

    class Meta:
        verbose_name = "Circuit Data"
        verbose_name_plural = "Circuit Data"
        ordering = ('showroom_config_data__location',)

    def __unicode__(self):
        return '%s | %s | %s | %s' % (self.showroom_config_data.location, self.provider, self.service_type, self.ref_no)

class CircuitFiles(models.Model): 
    circuit_contract_data = models.ForeignKey(CircuitInfoData,verbose_name="Showroom",blank=True)  
    circuit_file = models.FileField(blank=True,upload_to=service_upload_path)     
    file_name = models.CharField(max_length=200,verbose_name="File Name")

    class Meta:
            verbose_name = "Circuit Files"
            verbose_name_plural = "Circuit Files"    
            ordering = ('circuit_contract_data',)

    def __unicode__(self):
        return self.file_name

Upvotes: 0

Views: 546

Answers (1)

solarissmoke
solarissmoke

Reputation: 31404

Looping through and comparing the objects is inefficient and unnecessary because you can query the foreign key relationship backwards like so:

{% for item in Circuits %}
    <tr class="{% cycle 'tr-1' 'tr-2' %}">  
        <td>{{ item.provider }}</td>
        <td>{{ item.service_type }}</td>
        <td>{{ item.circuit_speed }}</td>
        <td>
        {% for file in item.circuitfiles_set.all %}
            <a href ="{{ file.circuit_file.url }}" target="_blank">{{ file.file_name }}</a><br />
        {% endfor %}
        </td>
    </tr>
{% endfor %}

You can then drop the CircuitFiles from your context. You can also use the url property of a FileField so that you don't need to prepend MEDIA_ROOT.

Also as general convention in Python, only class names are camel cased - all other variables are lower cased. Following this convention makes it easier to distinguish between a class and an object of that class.

Upvotes: 4

Related Questions