issue-#104 calling URLs and commands (revoting)

Locked

Should we merge topic branch issue-#104a with master?

Yes
3
33%
No
4
44%
Abstain
2
22%
 
Total votes: 9

Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

issue-#104 calling URLs and commands (revoting)

Post by Ivan Denisov »

This is NEW voting about issue-#104 (including improvements made by Josef):
http://redmine.blackboxframework.org/issues/104
The discussion

We are voting for adding two new procedures Dialog.Open for opening of files and URLs and Dialog.Run to run external applications.

The realization with hooks is allowing easy to replace the realization for errors handling or for security issues.

Calling external applications will not break normal execution flow of BlackBox, because separate threads are used.

The suggested differences:
http://redmine.blackboxframework.org/pr ... 115335d51f

The BlackBox version for testing:
http://blackboxframework.org/unstable/i ... a1.455.zip
User avatar
DGDanforth
Posts: 1061
Joined: Tue Sep 17, 2013 1:16 am
Location: Palo Alto, California, USA
Contact:

Re: issue-#104 calling URLs and commands (revoting)

Post by DGDanforth »

Works well for me. Both Run and Open.
-Doug
Zinn
Posts: 476
Joined: Tue Mar 25, 2014 5:56 pm
Location: Frankfurt am Main
Contact:

Re: issue-#104 calling URLs and commands (revoting)

Post by Zinn »

To the interface I vote yes
to the implementation I vote no
- Helmut
Zinn
Posts: 476
Joined: Tue Mar 25, 2014 5:56 pm
Location: Frankfurt am Main
Contact:

Re: issue-#104 calling URLs and commands (revoting)

Post by Zinn »

I am sorry Ivan,
we must stop this voting again and go back to the design phase. The solution is not well-thought-out.
It is not possible to merge this current state to the master, because we get name discrepancy.
Dialog.Open via HostDialog.Open confuse with HostCmds.Open.
We should not use the name Open here and find another name.
To hurry make no sense.
- Helmut
Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

Re: issue-#104 calling URLs and commands (revoting)

Post by Ivan Denisov »

Zinn wrote:I am sorry Ivan.
You should not excuse. If many members think that this issue can not be adopted, we should together think how to improve it.
That is normal process. This is one of the last features to add before beta version release.

However I will wait 2-3 days. Maybe some of the members will change opinion, because this issue is really ready for been adopted.
Dialog.Open does not confuse me or others except Helmut, so this is only serious argument for the delay.
User avatar
Josef Templ
Posts: 2047
Joined: Tue Sep 17, 2013 6:50 am

Re: issue-#104 calling URLs and commands (revoting)

Post by Josef Templ »

I agree with Helmut's comment on the naming.
It should be renamed to OpenExternal.
But then also Run must be renamed to RunExternal. For symmetry!

Also the implementation of HostDialog.Start should be changed to
the way it was in BB 1.6. Exactly the same, not using Dialog.Run.
This procedure is for backwards compatibility and as such it should be unmodified.
Calling Dialog.Run instead would change its semantics because this is now
done asynchronously.

Until the changes are done I vote No.

- Josef
Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

Re: issue-#104 calling URLs and commands (revoting)

Post by Ivan Denisov »

Now there is no uncertainty about the results of the voting :) I am closing it.
Locked