Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0005824opensim[REGION] Specific OpenSim Modulepublic2011-12-15 06:372012-01-06 15:05
ReporterMichelle Argus 
Assigned To 
PrioritynoneSeverityfeatureReproducibilityN/A
Statuspatch feedbackResolutionopen 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0005824: [Patch] Reorganised RemoteAdmin Module
DescriptionAdded a Patch as Proposal to reorganise RemoteAdmin. The Patch groups RAdmin commands by command type. The c# function are now grouped using #region for easy easely navigation.

 As I am planing on adding many new commands, I feel it would be good to reorganise RAdmin prior to implementing new commands. Over time the commands have become mixed a bit.

This is just a proposal. No functions or commands have been changed, only the sort order in which the functions are listed in the code have changed.
Additional InformationThis Patch requires the Patch in http://opensimulator.org/mantis/view.php?id=5814 [^] to be implemented first...
TagsNo tags attached.
Git Revision or version number63fe673af1974e265c5d18dc95f963fd968afd26
Run ModeStandalone (1 Region) , Standalone (Multiple Regions) , Grid (1 Region per Sim) , Grid (Multiple Regions per Sim)
Physics EngineBasicPhysics
Script Engine
EnvironmentUnknown
Mono VersionNone
Viewer
Attached Filespatch file icon 001-[Patch] RemoteAdmin-Proposal-to.reorganise-commands-and-functions.patch [^] (153,728 bytes) 2011-12-15 06:37 [Show Content]

- Relationships

-  Notes
(0020559)
justincc (administrator)
2011-12-16 16:48

Looks fine to me, though I think in OpenSim we usually indent #region directives to the same level as the class.

I'm assuming this relies on the application of http://opensimulator.org/mantis/view.php?id=5814 [^]
(0020560)
Michelle Argus (reporter)
2011-12-16 17:34

yes, it relies on the other mantis.

i can change the #region to the same level... i chose it diffrently because its easier when scrolling through the code. But if its standard in OS, then i will change it when i update the patch...
(0020635)
justincc (administrator)
2012-01-06 15:05

Current patch no longer applies due to other changes.

- Issue History
Date Modified Username Field Change
2011-12-15 06:37 Michelle Argus New Issue
2011-12-15 06:37 Michelle Argus File Added: 001-[Patch] RemoteAdmin-Proposal-to.reorganise-commands-and-functions.patch
2011-12-15 06:38 Michelle Argus Status new => patch feedback
2011-12-16 16:48 justincc Note Added: 0020559
2011-12-16 17:34 Michelle Argus Note Added: 0020560
2012-01-06 15:05 justincc Note Added: 0020635


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker