Project

General

Profile

Actions

Bug #2828

open

Actor names inconsistent

Added by Kirsten Menger-Anderson almost 17 years ago. Updated almost 15 years ago.

Status:
In Progress
Priority:
Normal
Assignee:
Category:
actors
Target version:
Start date:
04/18/2007
Due date:
% Done:

0%

Estimated time:
Bugzilla-Id:
2828

Description

Actor names and parameter names are currently inconsistently formatted. They should all be camel cap/no space (e.g., DirectoryListing) With an initial cap for Actor names, and an initial lowercase for parameter names.


Related issues

Is duplicate of Kepler - Bug #2438: Actors to be RenamedResolvedChad Berkley05/16/2006

Actions
Actions #1

Updated by Chad Berkley almost 17 years ago

Changing all of the actor and parameter/port names within kepler is not feasible. Actors are created by many different people around the world and enforcing this would be more effort than it's worth, although I agree it's a good goal to have new actors follow a common naming scheme.

Actions #2

Updated by Christopher Brooks almost 17 years ago

Hmm, I disagree with marking this as WONTFIX. The issue is that actor
names should be camel cased and that Actors should have an initial capital
letter and parameters should start with lower case. My opinion is that
if an actor is shipped with Kepler, it should be made to follow the coding
convention. Consistent names help users find things and create a uniform
environment.

However, as I'm merely an observer here, I won't reopen this.

Actions #3

Updated by Matt Jones almost 17 years ago

I agree -- these should indeed be fixed, and the effort to do so is well worth it. We also should be establishing standard for future contributions to the kepler core that are explicit about these requirements. I've reopened this bug and targeted at the 1.0.0 release.

Actions #4

Updated by Dan Higgins over 16 years ago

  • Bug 2438 has been marked as a duplicate of this bug. ***
Actions #5

Updated by Redmine Admin about 11 years ago

Original Bugzilla ID was 2828

Actions

Also available in: Atom PDF