Justin Clark-Casey (justincc)
0d0d04c484
minor: Add more information to OSFunctionThreatLevel and clarify some text in using PARCEL_OWNER, PARCEL_GROUP_MEMBER, ESTATE_MANAGER, ESTATE_OWNER permission categories
2012-07-11 23:05:09 +01:00
SignpostMarv
952b3448a6
porting console commands from raw2sculpt 3.2
2012-07-11 23:05:01 +01:00
Justin Clark-Casey (justincc)
d53ef3b302
Add OSSL function osForceAttachToAvatarFromInventory()
...
This works like osForceAttachToAvatar() but allows an object to be directly specified from the script object's inventory rather than forcing it to be rezzed in the scene first.
Still only attaches objects to the owner of the script.
This allows one to bypass the complicated co-ordination of first rezzing objects in the scene before attaching them.
Threat level high.
2012-07-11 23:04:49 +01:00
Justin Clark-Casey (justincc)
48e88397a0
Use GetInventoryItem() in llRezAtRoot rather than iterating through a cloned dictionary
2012-07-11 23:04:42 +01:00
Justin Clark-Casey (justincc)
976190fe96
refactor: In llGetNotecardLine() use existing GetInventoryItem() rather than inspecting a clone of the TaskInventory dictionary that was not cloned thread-safe
2012-07-11 23:04:34 +01:00
Justin Clark-Casey (justincc)
1e025d6074
refactor: In llGetNumberOfNotecardLines() use existing GetInventoryItem() rather than inspecting a clone of the TaskInventory dictionary that was not cloned thread-safe
2012-07-11 23:04:27 +01:00
Justin Clark-Casey (justincc)
fffd2fe10d
refactor: use existing GetInventoryItem() in GetScriptByName(), itself renamed from ScriptByName()
2012-07-11 23:01:41 +01:00
Justin Clark-Casey (justincc)
cf8adbe007
refactor: In llGetInventoryType() use existing GetInventoryItem()
2012-07-11 23:01:33 +01:00
Justin Clark-Casey (justincc)
890f617e58
refactor: In llGetInventoryCreator() use existing GetInventoryItem()
2012-07-11 23:01:27 +01:00
Justin Clark-Casey (justincc)
6ff0e58db7
refactor: In llGetInventoryPermMask() use existing GetInventoryItem()
2012-07-11 23:00:57 +01:00
Justin Clark-Casey (justincc)
795443cff1
refactor: in llGetInventoryPermMask use existing GetInventoryItem()
2012-07-11 23:00:50 +01:00
Justin Clark-Casey (justincc)
8c7f511e55
refactor: In llRemoteLoadScriptPin() use existing GetInventoryItem()
2012-07-11 23:00:43 +01:00
Justin Clark-Casey (justincc)
0acac2f890
refactor: replace use of LSL_Api.GetTaskInventoryItem() with existing GetInventoryItem()
2012-07-11 23:00:36 +01:00
Justin Clark-Casey (justincc)
72554fc5b8
refactor: In llGetInventoryKey() use existing GetInventoryItem()
2012-07-11 23:00:28 +01:00
Justin Clark-Casey (justincc)
aa1442b07e
In llMessageLinked() use GetInventoryItems() rather than cloning TaskInventory directory
...
GetInventoryItems() returns a new list and so is equivalent, and creates this list under lock whereas Clone() is not thread-safe
2012-07-11 23:00:21 +01:00
Justin Clark-Casey (justincc)
e8a0ec1287
In llRequestInventoryData() use GetInventoryItems() rather than cloning TaskInventory directory
...
GetInventoryItems() returns a new list and so is equivalent, and creates this list under lock whereas Clone() is not thread-safe
2012-07-11 23:00:13 +01:00
Justin Clark-Casey (justincc)
3436be2046
refactor: In llCollisionSound() use existing GetInventoryItem() method rather than have it iterate through TaskInventory itself.
2012-07-11 23:00:07 +01:00
Justin Clark-Casey (justincc)
9a0bcb7750
refactor: In llRemoveInventory() use existing GetInventoryItem() method rather than have it iterate through TaskInventory itself.
2012-07-11 22:59:59 +01:00
Justin Clark-Casey (justincc)
218fe36f84
refactor: make llGiveInventory() use existing GetInventoryItem() method rather than iterate through TaskInventory itself.
2012-07-11 22:59:53 +01:00
Justin Clark-Casey (justincc)
07298c8b4f
refactor: rename Watchdog.WATCHDOG_TIMEOUT_MS to DEFAULT_WATCHDOG_TIMEOUT_MS to reflect what it actually is
2012-07-11 22:59:46 +01:00
Justin Clark-Casey (justincc)
d9f40d1ebc
minor: Add some method doc to HasGroupChanged and Schedule GroupForFull/PartUpdate() to indicate when region modules need to invoke them
2012-07-11 22:59:39 +01:00
Justin Clark-Casey (justincc)
eddff428ed
Removing unused handling of incoming create object by userID and itemID only.
...
It appears this was never actually used since attachments were rezzed in other code.
This was never available on remote simulator comms, only local.
2012-07-11 22:58:57 +01:00
Justin Clark-Casey (justincc)
61aaa10460
Remove code listed for removal in 0.7.3 that handled script restart for incoming attachments from pre-fatpack regions (versions of OpenSimulator more than a year old)
2012-07-11 22:58:50 +01:00
Justin Clark-Casey (justincc)
e1d6929e61
Move update of the final optional ODE total frame stat inside the OdeLock rather than outside to avoid a very occasional race condition with the stat collection thread
2012-07-11 22:58:37 +01:00
Justin Clark-Casey (justincc)
6b87873aa7
refactor: rename _collisionEventPrim to m_collisionEventActors and _collisionEventPrimChanges to m_collisionEventActorsChanges to reflect their actual contents.
...
These dictionaries handle all actor types, not just physical prims.
2012-07-11 22:58:29 +01:00
Justin Clark-Casey (justincc)
f690acbfb6
Fix a regression where we stopped removing avatars from collision event reporting on logout, rather than stopping clearing their collision events.
...
This occurred in b18c8c8
(Thu May 17 2012).
This was a cause of very occasional race conditions and likely memory leakage as clients came and went from the region.
2012-07-11 22:58:09 +01:00
Justin Clark-Casey (justincc)
f49c850269
If a link points to a non-existing item in FetchInventory caps, then don't try to add it to the return data rather than suffering an exception later on
2012-07-11 22:58:01 +01:00
Justin Clark-Casey (justincc)
70f85af75b
Add IScene.Name for code clarity to replace the RegionInfo.RegionName used in many, many log messages.
2012-07-11 22:57:53 +01:00
Justin Clark-Casey (justincc)
663b0cc681
Change AttachmentsModule.DetachSingleAttachmentToInv() to accept a SOG directly instead of an item ID to then shuffle through attachments, saving CPU busywork.
...
Almost all callers already had the sog to hand.
Still checking that it's really an attachment, but now by inspecting SOG.AttachedAvatar
2012-07-11 22:57:39 +01:00
Justin Clark-Casey (justincc)
733a8c9f89
Rather than iterating through all SOGs in the scene looking for the one that matches out fromItemID on detach, go through the agent's attachment sog list instead.
2012-07-11 22:57:31 +01:00
Justin Clark-Casey (justincc)
25109c8e4d
Remove code that tried to delete an attachment back to inventory if RezSingleAttachmentFromInventoryInternal() returned null.
...
null would only ever be returned if the item couldn't be located within inventory and this would happen immediately.
In this case, derezzing wouldn't work anyway since there is no item to derez.
2012-07-11 22:57:22 +01:00
Justin Clark-Casey (justincc)
51724128bf
If rest of first line after colon is blank then still warn about running in XEngine if engine specified does not exist.
...
This is to take account of situations where the user was intending to specify a script engine using colon using its default language.
This probably generates few false positive as scripts are less likely to end a first line colon with a comment for other purposes.
2012-07-11 22:57:12 +01:00
Justin Clark-Casey (justincc)
461831a65e
Avoid reporting false positives when a colon is in a comment in the first line of a script where the user was not trying to select a different script engine.
...
This works by only posting the "Selected engine unavailable" message if we're falling back on XEngine and the language is one handled by XEngine.
In cases where the language is not handled or not allowed, the user will still be notified by the later compiler error.
This avoids the overwhelming majority of false positives where the first line contains a : for other reasons (e.g. source control systems, vim settings, etc.)
Ultimately, I think it would be better to detect script language/engine with a mechanism that didn't just rely on : detection (e.g like #! in unix scripts).
2012-07-11 22:56:59 +01:00
Justin Clark-Casey (justincc)
c6fa09c3af
minor: reuse colon index calculation in XEngine.OnRezScript. The index if a colon is found on the first line will always be the same as for the whole script.
2012-07-11 22:56:45 +01:00
Justin Clark-Casey (justincc)
9ccf56eaae
Fix issue in InventoryArchiveTestCase where it didn't call down to OpenSimTestCase.SetUp()
2012-06-27 21:52:23 +01:00
Justin Clark-Casey (justincc)
ea9b0794e0
refactor: Move ScenePresence <-> AgentData attachments copying code into AttachmentsModule.
2012-06-27 21:52:16 +01:00
Justin Clark-Casey (justincc)
96191241be
Automatically disable log4net before each regression test so that logging is confined to a single test if it's turned on.
...
This involves making test classes inherit from a common OpenSimTestCase.
This will be applied to more classes as required.
2012-06-27 21:52:08 +01:00
Justin Clark-Casey (justincc)
779f0ede49
Fix output for help on some object region console commands
2012-06-27 21:51:54 +01:00
Justin Clark-Casey (justincc)
e511106d8d
Show region positions in "show regions" robust console command
2012-06-27 21:51:48 +01:00
Justin Clark-Casey (justincc)
37f22e2946
minor: correct GridService "show regions" cibsike cinnabd usage statement
2012-06-27 21:51:40 +01:00
Justin Clark-Casey (justincc)
b802906827
refactor: Remove unnecessary AttachmentModuleTests.m_userId in favour of local variables
2012-06-27 21:51:34 +01:00
Justin Clark-Casey (justincc)
c6fb5f0fe4
refactor: Use local attachment module variables instead of global m_attMod.
...
This also avoids confusion between tests where one sets up m_attMod and another accidentally uses it after failing to set one up itself.
2012-06-27 21:51:27 +01:00
Justin Clark-Casey (justincc)
8909e70fc3
refactor: make m_presence a local variable in all AttachmentsModuleTests since it doesn't need to be global and some tests set up more than one sp
2012-06-27 21:51:19 +01:00
Justin Clark-Casey (justincc)
78fcee1f4b
Add regression test to check that attachments in source region are deleting when an agent teleports to a neighbouring region
2012-06-27 21:51:12 +01:00
Justin Clark-Casey (justincc)
3106a0f25e
If crossing attachments into another region pre-fatpack, clone objects before changing properties to avoid hud display race condition with update threads.
...
This matches behaviour in fatpack crossing, where attachments are cloned before their properties are changed.
This only applies to crossings to simulators running code released before April 2011.
2012-06-27 21:51:03 +01:00
Justin Clark-Casey (justincc)
19c2f08a8b
Add much easier ConsoleDisplayTable AddColumn() and AddRow() methods.
...
Use these for new "show regions" command rather than old cumbersome stuff.
2012-06-27 21:50:52 +01:00
Justin Clark-Casey (justincc)
005ca12a99
Add "show regions" console command to ROBUST to show all regions currently registered.
...
Command is not added in standalone, which has its own version of "show regions" that can also show estate name
2012-06-27 21:50:44 +01:00
Justin Clark-Casey (justincc)
02fe1a676e
Add "show region at" command to grid service to get the details of a region at a specific location. "show region" command becomes "show region name" to disambiguate
...
This is the same format as used by "show object name", etc.
"deregister region" also becomes "deregister region id"
2012-06-27 21:50:38 +01:00
Justin Clark-Casey (justincc)
03fc8cf155
minor: update currently commented out log message at top of AvatarFactoryModule.SetAppearance() for future use
2012-06-27 21:50:30 +01:00
Justin Clark-Casey (justincc)
8dedd91961
When attachments are being saved and deleted for a closing root agent, delete first to avoid a hud race condition with update threads.
...
If delete doesn't occur first then the update thread can outrace the IsAttachment = false necessary to save attachments and send hud artifacts to other viewers.
2012-06-27 21:50:22 +01:00