Ticket #2470 (closed bug: obsolete)

Opened 10 years ago

Last modified 9 years ago

applet-not-displaying-binary-resources - At the moment the applet has problems displaying binary resources.

Reported by: todor Owned by: mira
Priority: major Milestone: X3
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort:
Importance: 80 Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: todor Design_owners: mira
Imp._owners: mira Test_owners:
Analysis_reviewers: Changelog: Changelog
Design_reviewers: pap Imp._reviewers: deyan, todor, pap
Test_reviewers: Analysis_score: 0
Design_score: 3.5 Imp._score: 3.5
Test_score: 0

Description

Only text and solid color background can be viewed in the applet, it currently doesn't support any binary resources.

Attachments

appletBinaries.patch (26.9 KB) - added by mira 10 years ago.

Change History

comment:1 Changed 10 years ago by todor

  • Status changed from new to s1b_analysis_finished

comment:2 Changed 10 years ago by mira

  • Design_owners set to mira
  • Status changed from s1b_analysis_finished to s2a_design_started
  • Imp._owners set to mira

comment:3 Changed 10 years ago by mira

  • Status changed from s2a_design_started to s2b_design_finished

There is one more issue related to this task. When a connection is disconnected the locator for the server url is closed. This means that all books opened for this location are going to be closed no matter of their session id.

When removing a connection you should not close the locator, but only the TopAccess related to this connection. Make a method in the BaseLocator closeTopAccess(CacheKey) and call it when removing a connection. The ServerResourceLocator is created by a façade and url and the façade could be acquired by server url. They should be cached and managed by serverUrl:

  • The map in the provider should be <String, ServerResourceLocator >
  • When adding a connection just check if it is to new url and if needed add it with corresponding value – null.
  • When removing a connection close the TopAccess useing a CacheKey with its sessionid.
  • Make another method that can be used just to add a handled server url (so we don`t need a ServerConnection).

Extract static method getFcade(String serverUrl) from the ServerConnection façade() property and use it to make the new ServerResourceLocators by a server url only.

When the reader(or applet) receives initial String pointing to a server resource it should look like this:
“{book address}|{server address}|{session id}”

– The {book address} is the actual ref to be opened.

  • The {server address} is the server url to be added to the handled ones by the locator. Now we have only one locator and the MainFuncServersModule referees to it directly (this is not a good idea but the current implementation of the connections palette requires it)
  • The {session id} is used to make appropriate AccessOptions to open the book from the server with the AppLocator of the main window.

Since the method checkStartUpBook() should return both the bookRef and the accessOptions make a container class ResourceAccessInfo. It should just encapsulate ResourceRefR4 and AccessOptions.

comment:4 Changed 10 years ago by mira

  • Status changed from s2b_design_finished to s2c_design_ok

comment:5 Changed 10 years ago by mira

  • Owner set to mira
  • Status changed from s2c_design_ok to s3a_implementation_started

comment:6 Changed 10 years ago by mira

  • Status changed from s3a_implementation_started to s3b_implementation_finished

The implementation is done according to the design. I added a patch to the ticket. The implementation can be tested with the applet at 'straxil.asteasolutions.net/applet-demo'.

Changed 10 years ago by mira

comment:7 Changed 10 years ago by pap

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Design_score changed from 0 to 3.5
  • Design_reviewers set to pap
  • Imp._score changed from 0 to 3.5
  • Imp._reviewers set to deyan, todor, pap
  • Committed in [9226].
  • The same class addHandledServerUrl. Wrtie TODOs in the form TODO text --mira@2010-06-08. It sometimes helps ;)
  • I am not sure that this TODO is really sensible. Because in this module you provide an extension that you modify.
  • I somewhat dislike the dependency to the main.func.servers module :)
  • NO ANALYSIS REVIEW OR SCORE!!!

comment:8 Changed 9 years ago by meddle

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to obsolete

Closing all the tickets before M Y1

Note: See TracTickets for help on using tickets.