Skip to content

Commit adc83ef

Browse files
committed
orp-plugin: Fix handling of Names.
1 parent 322fff0 commit adc83ef

File tree

1 file changed

+27
-23
lines changed

1 file changed

+27
-23
lines changed

Diff for: orp-plugin/src/main/scala/orp/plugin/components/OrpComponent.scala

+27-23
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ private[components] trait OrpComponent extends PluginComponent with Transform wi
283283

284284
def actionMultiDef(counterRoleName: String)(action: String) = {
285285

286-
val counterRoleNameDecapitalized = name.plural(name.decapitalize(counterRoleName))
286+
val counterRoleNameDecapitalized = name.plural(name.decapitalize(counterRoleName).toString)
287287

288288
val vparamss = List(
289289
List(
@@ -296,7 +296,7 @@ private[components] trait OrpComponent extends PluginComponent with Transform wi
296296

297297
val rhs = fn(Ident(counterRoleNameDecapitalized), nme.foreach, Ident(actionDef))
298298

299-
DefDef(NoMods, name.plural(actionDef), Nil, vparamss, UNIT, rhs)
299+
DefDef(NoMods, name.plural(actionDef.toString), Nil, vparamss, UNIT, rhs)
300300
}
301301

302302
def replaceDef(counterRoleName: String)(multi: Boolean) = {
@@ -324,7 +324,7 @@ private[components] trait OrpComponent extends PluginComponent with Transform wi
324324
}
325325

326326
def clearDef(counterRoleName: String) = {
327-
val rhs = fn(thiz, name.plural(name.actionDef(RemovePrefix, counterRoleName)), Ident(name.getDef(counterRoleName)))
327+
val rhs = fn(thiz, name.actionDefPlural(RemovePrefix, counterRoleName), Ident(name.getDef(counterRoleName)))
328328
DefDef(NoMods, name.clearDef(counterRoleName), Nil, List(Nil), UNIT, rhs)
329329
}
330330

@@ -406,17 +406,17 @@ private[components] trait OrpComponent extends PluginComponent with Transform wi
406406
val counterClassNameDecapitalized = name.decapitalize(counterClassName)
407407
val mods = overrideIfNeeded(counterClassName, counterRoleName)
408408
DefDef(mods, name.actionDef(action, counterClassName), Nil,
409-
List(List(ValDef(NoMods, counterClassNameDecapitalized, Ident(stringToTypeName(counterClassName)), EmptyTree))),
409+
List(List(ValDef(NoMods, counterClassNameDecapitalized, Ident(newTypeNameCached(counterClassName)), EmptyTree))),
410410
UNIT, fn(create.zuper, name.actionDef(action, counterRoleName), Ident(counterClassNameDecapitalized)))
411411
}
412412

413413
def actionMultiWrapperDef(counterClassName: String, counterRoleName: String)(action: String) = {
414-
val counterClassNameDecapitalized = name.plural(name.decapitalize(counterClassName))
414+
val counterClassNameDecapitalized = name.plural(name.decapitalize(counterClassName).toString)
415415
val mods = overrideIfNeeded(counterClassName, counterRoleName)
416-
DefDef(mods, name.plural(name.actionDef(action, counterClassName)), Nil,
416+
DefDef(mods, name.actionDefPlural(action, counterClassName), Nil,
417417
List(List(ValDef(NoMods, counterClassNameDecapitalized,
418-
AppliedTypeTree(Ident(Traversable), List(Ident(stringToTypeName(counterClassName)))), EmptyTree))),
419-
UNIT, fn(create.zuper, name.plural(name.actionDef(action, counterRoleName)), Ident(counterClassNameDecapitalized)))
418+
AppliedTypeTree(Ident(Traversable), List(Ident(newTypeNameCached(counterClassName)))), EmptyTree))),
419+
UNIT, fn(create.zuper, name.actionDefPlural(action, counterRoleName), Ident(counterClassNameDecapitalized)))
420420
}
421421

422422
def replaceWrapperDef(counterClassName: String, counterRoleName: String)(multi: Boolean) = {
@@ -425,7 +425,7 @@ private[components] trait OrpComponent extends PluginComponent with Transform wi
425425
val crn: String = if (multi) name.plural(counterRoleName) else counterRoleName
426426

427427
def createVparam(classifier: String) = {
428-
val ccnrs = Ident(stringToTypeName(counterClassName))
428+
val ccnrs = Ident(newTypeNameCached(counterClassName))
429429
val ccnatt = AppliedTypeTree(Ident(Traversable), List(ccnrs))
430430
val tpt = if (multi) ccnatt else ccnrs
431431

@@ -466,35 +466,39 @@ private[components] trait OrpComponent extends PluginComponent with Transform wi
466466
private val PluralSuffix = "s"
467467
private val RoleSuffix = "Role"
468468

469-
def othersVal(name: String): TermName = {
470-
decapitalize(name) + PluralSuffix
469+
def othersVal(name: String) = {
470+
newTermNameCached(decapitalize(name) + PluralSuffix)
471471
}
472472

473-
def actionDef(actionName: String, name: String): TermName = {
474-
actionName + capitalize(name)
473+
def actionDef(actionName: String, name: String) = {
474+
newTermNameCached(actionName + capitalize(name))
475475
}
476476

477-
def clearDef(name: String): TermName = {
478-
ClearPrefix + capitalize(name) + PluralSuffix
477+
def actionDefPlural(actionName: String, name: String) = {
478+
newTermNameCached(plural(actionDef(actionName, name).toString))
479479
}
480480

481-
def getDef(name: String): TermName = {
482-
GetPrefix + capitalize(name) + PluralSuffix
481+
def clearDef(name: String) = {
482+
newTermNameCached(ClearPrefix + capitalize(name) + PluralSuffix)
483483
}
484484

485-
def priActionDef(actionName: String, name: String): TermName = {
486-
PriPrefix + capitalize(actionName) + capitalize(name)
485+
def getDef(name: String) = {
486+
newTermNameCached(GetPrefix + capitalize(name) + PluralSuffix)
487487
}
488488

489-
def withRoleSuffix(name: String): TypeName = {
490-
name + RoleSuffix
489+
def priActionDef(actionName: String, name: String) = {
490+
newTermNameCached(PriPrefix + capitalize(actionName) + capitalize(name))
491+
}
492+
493+
def withRoleSuffix(name: String) = {
494+
newTypeNameCached(name + RoleSuffix)
491495
}
492496

493497
def decapitalize(name: String) = {
494-
Introspector.decapitalize(name)
498+
newTermNameCached(Introspector.decapitalize(name))
495499
}
496500

497-
def plural(name: String): TermName = {
501+
def plural(name: String) = {
498502
name + PluralSuffix
499503
}
500504

0 commit comments

Comments
 (0)