taylor      2004/10/21 11:41:46

  Modified:    portal/src/java/org/apache/jetspeed/profiler/impl
                        ProfilerValveImpl.java
  Log:
  simplify and abstract profile processing in profiler valve
  
  Revision  Changes    Path
  1.18      +15 -8     
jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/profiler/impl/ProfilerValveImpl.java
  
  Index: ProfilerValveImpl.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/profiler/impl/ProfilerValveImpl.java,v
  retrieving revision 1.17
  retrieving revision 1.18
  diff -u -r1.17 -r1.18
  --- ProfilerValveImpl.java    15 Oct 2004 22:58:43 -0000      1.17
  +++ ProfilerValveImpl.java    21 Oct 2004 18:41:46 -0000      1.18
  @@ -73,23 +73,30 @@
               // get profiler locators for request subject/principal using the 
profiler
               Subject subject = request.getSubject();
               if (subject == null)
  +            {
                   throw new ProfilerException("Missing subject for request: " + 
request.getPath());
  +            }
               Principal principal = SecurityHelper.getBestPrincipal(subject, 
UserPrincipal.class);
               if (principal == null)
  +            {
                   throw new ProfilerException("Missing principal for request: " + 
request.getPath());
  -            String [] locatorNames = 
profiler.getLocatorNamesForPrincipal(principal);
  -            if ((locatorNames == null) || (locatorNames.length == 0))
  -                locatorNames = new String[]{ProfileLocator.PAGE_LOCATOR};
  -            Map locators = (Map) new HashMap(16);
  -            for (int i = 0; (i < locatorNames.length); i++)
  -                locators.put(locatorNames[i], 
profiler.getProfile(request,locatorNames[i]));
  +            }
   
  +            // get all locators for the current user
  +            Map locators = profiler.getProfileLocators(request, principal);
  +            if (locators.size() == 0)
  +            {
  +                locators.put(ProfileLocator.PAGE_LOCATOR, 
profiler.getProfile(request, ProfileLocator.PAGE_LOCATOR));
  +            }
  +            
               // get profiled page context using the profiler and page manager
               ProfiledPageContext profiledPageContext = 
profiler.createProfiledPageContext(locators);
               pageManager.computeProfiledPageContext(profiledPageContext);
               if (profiledPageContext.getPage() == null)
  +            {
                   throw new NodeNotFoundException("Unable to profile request: " + 
request.getPath());
  -
  +            }
  +            
               // set request page and profile locator
               request.setPage(profiledPageContext.getPage());
               request.setProfileLocators(profiledPageContext.getLocators());
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to