php7 methods with returnType not imported in classes

[flexjoly]

Hi,

I have many function/methods with a returntype specified, as is possible in php7.

Code: [Select]

class test{
  protected function myTest():bool{
    // some coding
  }
}
But these methods are not imported by reverse engineering. Anything else seems to work nicely from php7.
I found the code-templates and the grammar framework. Changing the operation-template does not help. I tried finding the php-grammar file, but I guess the internal grammar-files cannot be changed in the grammar-framework.

Can the returntype be added for EA class diagrams?

Thanks in advance!
flexJoly

[flexjoly]

Hello,

Is there any change that this get fixed in the near future?
That the returntype is not recognized is no problem, but that the class is not imported at all thát is a problem.
Can this please be fixed?

We have the corporate version. Is it possible and not too difficult to script around it?
Does anyone have a scripting solution for this? (I am really a newby on scripting in EA)

@geertbellekens > do you have something for this?

Thanks in advance!
Lydia

[Paolo F Cantoni]

Hi Lydia,

NOTHING gets fixed without a formal request using one of the links at the bottom.

Even then, it seems to be a lottery, but certainly, not request/ report, no fix!

Paolo

[flexjoly]

Hi Paolo,

Thanks for pointing this out.

I did another feature request a while ago.... except from the confirmation mail you hear nothing on it...
So i did not thing of that anymore.

But on your advice, I looked at it again and see that you can post a bug report as registered user. So I did that now on this. I hope that EA will at least respond.

Greetz, Lydia

[Geert Bellekens]

Yes, feature requests usually disappear in the Sparx void never to be heard of again.
But sometimes they do implement these, unfortunately without letting you know.

So what I do is check the release notes with every release to see if any of my feature requests made it.

Bug reports are a bit different. Usually I get a confirmation saying "yes, we confirm it's a bug and it is planned to be fixed some day in the future"
(or "no this is by design, we are not willing to change that just because you seem to think that 1 + 1 = 2)

Sometimes you even get a message saying "we fixed the bug you reported in version x", but that is rare.

Geert

[Eve]

Copy this code into a text/xml file.

Code: [Select]

<MDG.Technology version="1.0">
    <Documentation id="PHP7" name="PHP 7" version="1.0" notes="Temporary and partial support for PHP 7."/>
    <CodeModules>
        <CodeModule language="PHP" notes="">
            <CodeOptions>
        <CodeOption name="PrependGrammarDefinitions">
        <![CDATA[
<paramType> ::= <actualType:TYPE>
| <"":TYPE="var">
| <fail>;
<functionDec:DECLARATION>
::= <funcModifier>* <scope> <funcModifier>* "function" <identifier:NAME> <params> ":" <actualType>
| <funcModifier>* <scope> <funcModifier>* "function" <functionType> <identifier:NAME> <params>
| <identifier:NAME> <params>
| <fail>;
                    ]]>
        </CodeOption>
        <CodeOption name="AppendGrammarDefinitions">
        <![CDATA[
<nullableType> ::= "?" <qualifiedName>;
<actualType> ::= <variadicParamType>
| <nullableType>;
                    ]]>
        </CodeOption>
            </CodeOptions>
        </CodeModule>
     </CodeModules>
</MDG.Technology>
Import it as an MDG technology.

In v15 I think you also need to create a custom perspective containing this technology.

[flexjoly]

Hi Eve,

Thank you so much for your effort!
Sorry I did not see this sooner.

I tried to import your xml as mdg. And did also:
- select 'php 7' as required (https://sparxsystems.com/enterprise_architect_user_guide/15.0/modeling/specify_reqd_mdg_technologies.html)
- php7 is also shown in configure > resources > mdg techn.
- set it to active here: https://sparxsystems.com/enterprise_architect_user_guide/15.0/modeling/manage_mdg_technologies.html
- restarted EA several times
- made a new test project
- (re-)imported/synchronized some php7
- language of the package is 'PHP'

In v15 I think you also need to create a custom perspective containing this technology.

I overlooked this at first and I have no experience here so far with this. 
But I tried to make one,  then I needed to choose a model (wizard) to make a diagram. So does this have to do with importing source?

So far I cannot get it to work.
The classes are imported (from a directory), but the methods with a returnType are not.
I hope you can help me here again.

Thanks in advance, Lydia


NB! Until now, I did not hear anything on my bug report.

[flexjoly]

Hi,

I also tried again to change the grammer for 'Operation Declaration' like:

Code: [Select]

%PI=" "%
%if genOptPHPVersion >= "5.0"%
%if elemType=="Interface"%
public
%else%
%opAbstract == "T" ? "abstract" : ""%
%CONVERT_SCOPE(opScope)%
%endIf%
%endIf%
%opStatic=="T" ? "static" : ""%
%if genOptPHPVersion >= "5.0"%
%opTag:"final" != "" ? "final" : ""%
%endIf%
function
%PI=""%
%opName%(%list="Parameter" @separator=", "%)
%PI=""%
:
%opReturnType=="var*" ? "&" : ""%

I tried different things with 'opReturnType', but so far no luck.

Any suggestions?

Thanks in advance, Lydia

[Eve]

Changing the code generation templates won't change the import behavior at all. Neither will setting required or active, but required could prevent other people from accessing your model.

The behavior with perspectives was not intentional, and I believe it has been corrected in the latest builds. So the easiest approach is updating to the latest build.

In the perspectives menu, select manage perspectives. On the second page create a new perspective and select the php7 technology. Save it and you're done. As long as you are in 'All' or your new technology when first importing PHP in an EA session it should take effect.

[flexjoly]

Hi Eve,

I tried this again, but it is still not working
Methods with type-hinting return, are not imported into the class-elements.

EA is not an cheap application.... (at least for our small company). And with the much praised MDG, why is it so hard for sparx to update the php to version 7  
Can something please be done on this?!?

It is that other uml-tools are worse.... else....

Sorry, I am very frustrated after 2 days with EA and not getting done simple things (also struggling with namespaces).

Greetz, flexjoly

[qwerty]

Well, you "can" get EA to work. If you know how (can be tricky to, well, just vomiting awful work arounds). And over time it's getting worse. EA suffers from feature creep. Rather than getting bugs fixed they add new stuff with even more bugs.

Now, for other tools: They are differently worse. Many are much more expensive and do not offer the expected added value :-/ The world is a sad place.

q.