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

Technology Process - Missing Sprite #5

Closed
zimchaa opened this issue Mar 19, 2019 · 2 comments
Closed

Technology Process - Missing Sprite #5

zimchaa opened this issue Mar 19, 2019 · 2 comments

Comments

@zimchaa
Copy link
Contributor

zimchaa commented Mar 19, 2019

The technology-process while a valid Archimate item doesn't exist as a valid sprite, per this diagram (listsprite):

listsprite

It's possible to use process however to recreate a valid looking entity:

Working Technology Sprite

PlantUML code:

@startuml
!includeurl https://raw.githubusercontent.com/ebbypeter/Archimate-PlantUML/master/Archimate.puml

title Archimate Overview

'LAYOUT_AS_SKETCH
'LAYOUT_LEFT_RIGHT
'LAYOUT_TOP_DOWN

Grouping(B_Group,"Business"){
        Business_Process(BProcess, "Business Process")
}

Grouping(T_Group, "Technology"){

    Technology_Process(TProcess, "Technology Process")
archimate #TECHNOLOGY "Working Technology Process" <<process>>
archimate #TECHNOLOGY "Non-Working Technology Process" <<technology-process>>

}
@enduml

It could be addressed by a simple change in the include define for that macro, (line 104) from:

!define Technology_Process(e_alias, e_label) archimate #TECHNOLOGY "e_label" <<technology-process>> as e_alias

to:

!define Technology_Process(e_alias, e_label) archimate #TECHNOLOGY "e_label" <<process>> as e_alias

I know it doesn't look as nice, but it would fix an error in one of your samples - https://github.com/ebbypeter/Archimate-PlantUML/blob/master/samples/Archimate-Overview.wsd without need to change the code in that sample however.

zimchaa added a commit to zimchaa/Archimate-PlantUML that referenced this issue Mar 19, 2019
Change the Technology Process sprite from ``technology-process`` to ``process`` - this shows the sprite correctly without changing the way that existing PlantUML code that uses this library is used.
@smeagol74
Copy link
Contributor

I just submitted a pull request to plantuml that fixes some missed icons and colors for archimate plantuml/plantuml#205 so if it will be merged this issue will be fixed authomatically.

ebbypeter added a commit that referenced this issue May 3, 2020
Update Technology_Process Macro - per Issue #5
@Potherca
Copy link
Member

@smeagol74 plantuml/plantuml#205 has been merged, is it okay to close this issue?

Potherca pushed a commit to potherca-contrib/Archimate-PlantUML that referenced this issue Jul 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants