JavaScript code should not be requesting XHTML representations from the web service

Bug #490826 reported by Māris Fogels
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Tim Penhey

Bug Description

JavaScript in Launchpad should not be requesting the default XHTML representation from the webservice, just so it can cherry-pick the specific attributes that the code needs. Instead, the code should request just the fields it needs.

Related branches

Māris Fogels (mars)
tags: added: tech-debt
tags: added: javascript
Revision history for this message
Björn Tillenius (bjornt) wrote : Re: [Bug 490826] [NEW] JavaScript code should not be requesting XHTML representations from the web service

On Tue, Dec 01, 2009 at 01:57:49PM -0000, Māris Fogels wrote:
> Public bug reported:
>
> JavaScript in Launchpad should not be requesting the default XHTML
> representation from the webservice, just so it can cherry-pick the
> specific attributes that the code needs. Instead, the code should
> request just the fields it needs.

I assume the plan is to remove the default XHTML representation as well,
so that people aren't tempted to use it?

Revision history for this message
Māris Fogels (mars) wrote :

2009/12/2 Björn Tillenius <email address hidden>:
> On Tue, Dec 01, 2009 at 01:57:49PM -0000, Māris Fogels wrote:
>> Public bug reported:
>>
>> JavaScript in Launchpad should not be requesting the default XHTML
>> representation from the webservice, just so it can cherry-pick the
>> specific attributes that the code needs.  Instead, the code should
>> request just the fields it needs.
>
> I assume the plan is to remove the default XHTML representation as well,
> so that people aren't tempted to use it?
>

Correct, the default should be removed. Pages can create their own all-in-one representation if they want.

Curtis Hovey (sinzui)
Changed in launchpad-foundations:
importance: Undecided → Low
Revision history for this message
Tim Penhey (thumper) wrote :

What we are doing now with lazr.restful 0.17.0 is to get json with included lp_html. This allows us to keep the client cache up to date.

Changed in launchpad:
status: Triaged → In Progress
assignee: nobody → Tim Penhey (thumper)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
Changed in launchpad:
milestone: none → 11.03
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Tim Penhey (thumper)
tags: added: qa-ok
removed: qa-needstesting
Changed in launchpad:
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.