/products/gnomebaker/+translations has a legend but no chart

Bug #2718 reported by Matthew Paul Thomas
8
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Данило Шеган

Bug Description

When logged out, or when logged in as mpt,
https://launchpad.net/products/gnomebaker/+translations has a legend for a chart that's not there.

The problem occurs on launchpad.net, but not on demo.launchpad.net or staging.launchpad.net.

Bug 2919 may be a duplicate of this bug.
Bug 47781 may be related to this bug.

Christian Reis (kiko)
Changed in rosetta:
assignee: nobody → gabrielneuman
description: updated
description: updated
Revision history for this message
Gabriel Neuman (gabrielneuman) wrote :

Update needed.
I just tryied to reproduce it locally, but unlogged, logged as Sample Person or Foo Bar have the same efects: all show legend and chart.
Unfortunally on launchpad.net I got only the chart while unlogged.
MPT, can you reproduce the bug in your local branch?

Revision history for this message
Gabriel Neuman (gabrielneuman) wrote :

One more thing, locally I only tryied it on http://localhost:8086/products/netapplet/+translations
because all ohter products "have not yet been setup for translation through Rosetta" in the database...

Revision history for this message
Gabriel Neuman (gabrielneuman) wrote :

The following code takes me to conclusion that somehow mpt and unlooged users have this condition "view/languages" = FALSE.
The easyiest way to solve this is put the legend inside this div.
The right thing to do, would be checking why it is happening.
Could anyone suggest anything?
(/templates/object-pots.pt)
(refers to +chart and +rosetta-status-legend)

      <div tal:condition="view/languages"
           tal:repeat="potview view/templateviews"
           tal:omit-tag=""> <!-- PO Template Details -->

        <h2><span tal:replace="potview/context/title">Template Title</span></h2>

        <p tal:content="potview/description">
          Template description
        </p>

        <div tal:replace="structure potview/context/@@+chart" />

        <div align="right">
          [ <a tal:attributes="href potview/context/fmt:url">View Template
          & All Languages...</a> ]
        </div>

      </div>

      <div tal:replace="structure context/@@+rosetta-status-legend" />

Revision history for this message
Christian Reis (kiko) wrote :

I have a patch that fixes this in a tree of mine, I am going to try and merge it between today and monday.

Changed in rosetta:
assignee: gabrielneuman → kiko
Revision history for this message
Carlos Perelló Marín (carlos) wrote :

kiko, what happens with that merge? is it done?

Changed in rosetta:
status: New → PendingUpload
Revision history for this message
Carlos Perelló Marín (carlos) wrote :

Easy template fix.

Changed in rosetta:
status: Fix Committed → Fix Released
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

This has regressed.

Changed in rosetta:
status: Fix Released → Confirmed
description: updated
description: updated
description: updated
Revision history for this message
Christian Reis (kiko) wrote :

He's touched related code recently, help!

Changed in rosetta:
assignee: kiko → danilo
Changed in rosetta:
status: Confirmed → In Progress
Changed in rosetta:
status: In Progress → Fix Committed
Changed in rosetta:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.