{"action":"created","issue":{"url":"https://api.github.com/repos/tbeu/Modelica/issues/523","repository_url":"https://api.github.com/repos/tbeu/Modelica","labels_url":"https://api.github.com/repos/tbeu/Modelica/issues/523/labels{/name}","comments_url":"https://api.github.com/repos/tbeu/Modelica/issues/523/comments","events_url":"https://api.github.com/repos/tbeu/Modelica/issues/523/events","html_url":"https://github.com/tbeu/Modelica/issues/523","id":197517106,"number":523,"title":"Replaceable base class in MediaFix MSL 3.1 branch","user":{"login":"modelica-trac-importer","id":22202507,"avatar_url":"https://avatars.githubusercontent.com/u/22202507?v=3","gravatar_id":"","url":"https://api.github.com/users/modelica-trac-importer","html_url":"https://github.com/modelica-trac-importer","followers_url":"https://api.github.com/users/modelica-trac-importer/followers","following_url":"https://api.github.com/users/modelica-trac-importer/following{/other_user}","gists_url":"https://api.github.com/users/modelica-trac-importer/gists{/gist_id}","starred_url":"https://api.github.com/users/modelica-trac-importer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/modelica-trac-importer/subscriptions","organizations_url":"https://api.github.com/users/modelica-trac-importer/orgs","repos_url":"https://api.github.com/users/modelica-trac-importer/repos","events_url":"https://api.github.com/users/modelica-trac-importer/events{/privacy}","received_events_url":"https://api.github.com/users/modelica-trac-importer/received_events","type":"User","site_admin":false},"labels":[{"id":506770353,"url":"https://api.github.com/repos/tbeu/Modelica/labels/discussion","name":"discussion","color":"cccccc","default":false},{"id":506769720,"url":"https://api.github.com/repos/tbeu/Modelica/labels/L:Media","name":"L:Media","color":"0000aa","default":false}],"state":"open","locked":false,"assignee":{"login":"beutlich","id":14896695,"avatar_url":"https://avatars.githubusercontent.com/u/14896695?v=3","gravatar_id":"","url":"https://api.github.com/users/beutlich","html_url":"https://github.com/beutlich","followers_url":"https://api.github.com/users/beutlich/followers","following_url":"https://api.github.com/users/beutlich/following{/other_user}","gists_url":"https://api.github.com/users/beutlich/gists{/gist_id}","starred_url":"https://api.github.com/users/beutlich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beutlich/subscriptions","organizations_url":"https://api.github.com/users/beutlich/orgs","repos_url":"https://api.github.com/users/beutlich/repos","events_url":"https://api.github.com/users/beutlich/events{/privacy}","received_events_url":"https://api.github.com/users/beutlich/received_events","type":"User","site_admin":false},"assignees":[{"login":"beutlich","id":14896695,"avatar_url":"https://avatars.githubusercontent.com/u/14896695?v=3","gravatar_id":"","url":"https://api.github.com/users/beutlich","html_url":"https://github.com/beutlich","followers_url":"https://api.github.com/users/beutlich/followers","following_url":"https://api.github.com/users/beutlich/following{/other_user}","gists_url":"https://api.github.com/users/beutlich/gists{/gist_id}","starred_url":"https://api.github.com/users/beutlich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beutlich/subscriptions","organizations_url":"https://api.github.com/users/beutlich/orgs","repos_url":"https://api.github.com/users/beutlich/repos","events_url":"https://api.github.com/users/beutlich/events{/privacy}","received_events_url":"https://api.github.com/users/beutlich/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2016-12-25T22:21:04Z","updated_at":"2016-12-25T22:21:23Z","closed_at":null,"body":"**Reported by per&#x2e;ostlund on 7 Apr 2011 09:03 UTC**\nThe Modelica.Media.IdealGases.Common.FluidData.FluidConstants record in the MediaFix branch extends from Modelica.Media.Interfaces.PartialMedium.FluidConstants, which is replaceable. Section 7.1.4 in the Modelica 3.2 specification says that base classes must be transitively non-replaceable, which I understand means that it should not contain any replaceable elements on any level. However, I can not find any reason why the base class itself should not be allowed to be replaceable, but since Dymola gives a warning when they are I assume there is such a reason? \n\nIf base classes are not allowed to be replaceable then the MediaFix branch needs to be fixed. If they are allowed it might be a good idea to add this to the 7.1.4 section in the specification, to make it clearer what is and isn't allowed.\n\nP.S. There's no correct correct option to select for Version when adding a new issue if the issue concerns a branch of the MSL. Perhaps some generic branch option should be added?\n\n----\n\nMigrated-From: https://trac.modelica.org/Modelica/ticket/523"},"comment":{"url":"https://api.github.com/repos/tbeu/Modelica/issues/comments/269139631","html_url":"https://github.com/tbeu/Modelica/issues/523#issuecomment-269139631","issue_url":"https://api.github.com/repos/tbeu/Modelica/issues/523","id":269139631,"user":{"login":"modelica-trac-importer","id":22202507,"avatar_url":"https://avatars.githubusercontent.com/u/22202507?v=3","gravatar_id":"","url":"https://api.github.com/users/modelica-trac-importer","html_url":"https://github.com/modelica-trac-importer","followers_url":"https://api.github.com/users/modelica-trac-importer/followers","following_url":"https://api.github.com/users/modelica-trac-importer/following{/other_user}","gists_url":"https://api.github.com/users/modelica-trac-importer/gists{/gist_id}","starred_url":"https://api.github.com/users/modelica-trac-importer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/modelica-trac-importer/subscriptions","organizations_url":"https://api.github.com/users/modelica-trac-importer/orgs","repos_url":"https://api.github.com/users/modelica-trac-importer/repos","events_url":"https://api.github.com/users/modelica-trac-importer/events{/privacy}","received_events_url":"https://api.github.com/users/modelica-trac-importer/received_events","type":"User","site_admin":false},"created_at":"2016-12-25T22:21:23Z","updated_at":"2016-12-25T22:21:23Z","body":"**Comment by per&#x2e;ostlund on 8 Apr 2011 12:06 UTC**\nI found out that it's illegal for a base class to be replaceable in a normal extends, and in retrospect this is made clear in the specification. So the MediaFix branch needs to be fixed, since it's not legal for Modelica.Media.IdealGases.Common.FluidData.FluidConstants to extend Modelica.Media.Interfaces.PartialMedium.FluidConstants."}}
{"actor":{"display_login":"modelica-trac-importer"}}