oreolek/vantaMOO
1
0
Fork 0
Code Issues 14 Pull requests Packages Projects Releases Wiki Activity

Inherit RPCharacter from ClothedCharacter #30

Closed
oreolek wants to merge 2 commits from refs/pull/30/head into vanta
pull from: refs/pull/30/head
merge into: oreolek:vanta
Conversation 9 Commits 2 Files changed 2 +4 -4
oreolek commented 2023-01-11 07:26:02 +02:00 (Migrated from git.disroot.org)
Copy link

Multi-inheritance breaks get_display_things

This is a way to address #29

Multi-inheritance breaks `get_display_things` This is a way to address #29
lunacb commented 2023-01-11 08:45:19 +02:00 (Migrated from git.disroot.org)
Copy link

This doesn't fix the problem. Now ContribRPCharacter's methods just override ClothedCharacter's.

This doesn't fix the problem. Now `ContribRPCharacter`'s methods just override `ClothedCharacter`'s.
lunacb commented 2023-01-11 08:52:31 +02:00 (Migrated from git.disroot.org)
Copy link

I don't think there's a good solution to the problem that doesn't involve refactoring the contrib code.

I don't think there's a good solution to the problem that doesn't involve refactoring the contrib code.
oreolek commented 2023-01-11 08:54:30 +02:00 (Migrated from git.disroot.org)
Copy link

Now ContribRPCharacter's methods just override ClothedCharacter's.

Not quite. ContribRPCharacter defers to super's method if there's no a special RP code present, so it will use ClothedCharacter's code instead of default.

> Now `ContribRPCharacter`'s methods just override `ClothedCharacter`'s. Not quite. `ContribRPCharacter` defers to `super`'s method if there's no a special RP code present, so it will use `ClothedCharacter`'s code instead of default.
lunacb commented 2023-01-11 09:01:12 +02:00 (Migrated from git.disroot.org)
Copy link

oh, you're right.

I still think it's good to refactor more eventually, though. it still feels messy and unintuitive to me, and wouldn't scale well if more contrib code/features were to be added.

oh, you're right. I still think it's good to refactor more eventually, though. it still feels messy and unintuitive to me, and wouldn't scale well if more contrib code/features were to be added.
oreolek commented 2023-01-11 09:09:39 +02:00 (Migrated from git.disroot.org)
Copy link

I'm not a python dev so this is the best i can do for now.

I'm not a python dev so this is the best i can do for now.
lunacb commented 2023-01-11 12:11:51 +02:00 (Migrated from git.disroot.org)
Copy link

Sorry, I'm not trying to be accusatory! I'm not a python dev either.

Sorry, I'm not trying to be accusatory! I'm not a python dev either.
vantablack commented 2023-01-17 09:10:20 +02:00 (Migrated from git.disroot.org)
Copy link

i can't merge this

it says "This pull request has changes conflicting with the target branch.
typeclasses/characters.py"

i can't merge this it says "This pull request has changes conflicting with the target branch. typeclasses/characters.py"
oreolek commented 2023-01-17 14:54:35 +02:00 (Migrated from git.disroot.org)
Copy link

okay i'm gonna defer to @lunacb on this one because she already rewrote this code while working on pronouns.

okay i'm gonna defer to @lunacb on this one because she already rewrote this code while working on pronouns.
lunacb commented 2023-01-17 23:05:34 +02:00 (Migrated from git.disroot.org)
Copy link

I'll open a new PR. This one can be closed.

I'll open a new PR. This one can be closed.

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: oreolek/vantaMOO#30
No description provided.
Delete branch "refs/pull/30/head"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?