Bug 107708 - Remove the c.a.kernel.KernelHelper class
Summary: Remove the c.a.kernel.KernelHelper class
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Web Application Framework
Classification: Retired
Component: other
Version: nightly
Hardware: All
OS: Linux
medium
high
Target Milestone: ---
Assignee: ccm-bugs-list
QA Contact: Jon Orris
URL:
Whiteboard:
Depends On:
Blocks: 111508
TreeView+ depends on / blocked
 
Reported: 2003-10-22 11:04 UTC by Daniel Berrangé
Modified: 2007-04-18 16:58 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-09-02 17:24:06 UTC
Embargoed:


Attachments (Terms of Use)

Description Daniel Berrangé 2003-10-22 11:04:03 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 Galeon/1.2.9 (X11; Linux i686; U;) Gecko/20030314

Description of problem:
The c.a.kernel.KernelHelper class has been deprecated for several releases now,
with equivalent & better functionality available from KernelContext and
WebContext. It is about time it was removed altogether, particularly so we can
identify the few remaining places that use getCurrentUser().



Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. N/A
2.
3.
    

Additional info:

Comment 1 Daniel Berrangé 2003-11-07 15:50:22 UTC
I turns out that since we removed most of the config params from the
legacy kernel.Initializer, nothing is ever setting up the variables
used by getServerName(), getServerPort(), getHostname() methods of
KernelHelper. As a result all code that does still happen to be using
KernelHelper is getting "" & thus breaking in also sorts of subtle ways. 

We should either remove getServerName(), etc immediately forcing
people to switch the new APIs, or make the existing deprecated ones
proxies for corresponding methods in Web.getConfig() so that the
existing semantics are preserved.



Comment 2 Daniel Berrangé 2006-09-02 17:24:06 UTC
Closing old tickets



Note You need to log in before you can comment on or make changes to this bug.