Duplication Vs Abstraction

after-DRY.py

def can_user_act_on_document(user, document, action):
    if user.is_admin:
        return True
    if action == "view":
        return document.owner_id == user.id
    if action == "edit":
        return document.owner_id == user.id
    return False  # unknown action

# growing conditionals and nested logic

duplication.py

def can_user_view_document(user, document):
    return document.owner_id == user.id or user.is_admin

def can_user_edit_document(user, document):
    return document.owner_id == user.id

later.py

def can_user_act_on_document(user, document, action, context=None):
    if user.is_admin:
        return True

    if action == "view":
        if document.shared_with and user.id in document.shared_with:
            return True
        if document.status == "published" and document.is_public:
            return True
        return document.owner_id == user.id

    if action == "edit":
        if document.workspace_id in user.collaborating_workspaces:
            return True
        if context and context.get("override"):
            log_override(user, document, action)
            return True
        return document.owner_id == user.id

    log_permission_denied(user, document, action)
    return False