Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FLIP 210: propose an improvement to entitlement mapping syntax #210

Merged
merged 4 commits into from
Oct 24, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 43 additions & 0 deletions cadence/20231016-entitlement-mapping-syntax.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
---
status: draft
dsainati1 marked this conversation as resolved.
Show resolved Hide resolved
flip: NNN (set to the issue number)
dsainati1 marked this conversation as resolved.
Show resolved Hide resolved
authors: Daniel Sainati ([email protected])
sponsor: Daniel Sainati ([email protected])
updated: 2023-10-16
---

# FLIP NNN: Improve Entitlement Mapping Syntax
dsainati1 marked this conversation as resolved.
Show resolved Hide resolved

## Objective

This proposes a syntax change to make entitlement-mapped fields more visually distinct from entitled fields.

## Motivation

User feedback has indicated that entitlement mappings are too easy to confuse with entitlements,
and since one of them has default behavior for public access and the other prevents it,
it can be difficult to reason about what fields and functions are publicly available in the presence
of both entitled and mapped fields
dsainati1 marked this conversation as resolved.
Show resolved Hide resolved

## User Benefit

This will make it easier for users to distinguish between entitled members and mapped members.

## Design Proposal

Currently, an entitlement mapped field or an entitlement mapped reference is specified by using the
name of a mapping (e.g. `M`) in an `access` or `auth` modifier (e.g. `access(M)` or `auth(M)`).

Instead, we would require the keyword `mapping` to be present in the modifiers to indicate
that `M` is a mapping here, rather than an entitlement.
So the above modifiers would become `access(mapping M)` and `auth(mapping M)`

### Drawbacks

This makes the creation of entitlement mapped fields and functions 16 characters more verbose. I.e. a function
that was previously declared as `access(M) fun foo(): auth(M) &T` would instead have to be declared as
`access(mapping M) fun foo(): auth(mapping M) &T`, which is more verbose.

### Alternatives Considered

We have not discussed alternative syntactic changes, but are open to suggestions.