cortex-cms/cortex

View on GitHub

Showing 76 of 76 total issues

Unused block argument - error. You can omit the argument if you don't care about it.
Open

        m.failure do |error|

This cop checks for unused block arguments.

Example:

# bad

do_something do |used, unused|
  puts used
end

do_something do |bar|
  puts :foo
end

define_method(:foo) do |bar|
  puts :baz
end

Example:

#good

do_something do |used, _unused|
  puts used
end

do_something do
  puts :foo
end

define_method(:foo) do |_bar|
  puts :baz
end

TODO found
Open

    # TODO: this logic effectively gets called multiple times (slow?) - how do we optimize or cache the result?
Severity: Minor
Found in app/models/cortex/content_item.rb by fixme

TODO found
Open

        # TODO: iterate through, provide JSON of Widget parser for frontend

TODO found
Open

    # FieldItem and State Convenience Methods. TODO: move to concern? transactions?
Severity: Minor
Found in app/models/cortex/content_item.rb by fixme

TODO found
Open

      window.location = '/admin/dashboards'; // TODO: remove when we fully switch to React

TODO found
Open

    # TODO: the whole thing

TODO found
Open

        # TODO: implement & move import to ContentItem to mass-refresh all ContentItems for a ContentType

TODO found
Open

    def field_name(prefix, content_type) # TODO: extract

TODO found
Open

# TODO: ContentTypes, Decorators or Contracts themselves should inform the system what the correct Decorator is, not a hardcoded concern method

TODO found
Open

      # TODO: This needs to be in a plugin
Severity: Minor
Found in app/cells/cortex/index_cell.rb by fixme

TODO found
Open

      # TODO: This needs to be in a plugin
Severity: Minor
Found in app/cells/cortex/index_cell.rb by fixme

TODO found
Open

        # TODO: scope to tenant; allow tenant argument

TODO found
Open

    # TODO: Owned ContentItems, etc

TODO found
Open

      content_item.tenant = input.current_user.active_tenant # TODO: In future, grab from form/route, rather than current_user + perform authorization checks

TODO found
Open

      # TODO: This needs to be generic functionality
Severity: Minor
Found in app/cells/cortex/index_cell.rb by fixme

TODO found
Open

    # TODO: Owned ContentItems, etc
Severity: Minor
Found in app/graphql/cortex/types/user_type.rb by fixme

TODO found
Open

      # TODO: basic pagination

TODO found
Open

    # TODO: Extract to a concern
Severity: Minor
Found in app/models/cortex/content_type.rb by fixme

TODO found
Open

  -# TODO: remove conditional once fields are properly converted to React

TODO found
Open

        # TODO: DRY - this logic appears in other classes
Severity
Category
Status
Source
Language