include functions syntax #28

Closed
opened 2018-07-29 08:00:33 +02:00 by belliash · 5 comments
Owner

Aer Information

  • Aer Version (or commit ref): 48b2e9d1c6
  • Operating System: Linux
  • System Architecture (eg. arm, x86_64, ...): x86_64

Your problem description

Include functions [include(), include_once(), require(), require_once()] does not check for opening and closing bracket. They can be called both:

include('file.php');

and

include 'file.php';

As there is no limit of using including function and such instruction can be put anywhere, IMHO it should have syntax like standard function.

Expected results

include 'file.php'; should fail.

Current results

no error about wrong syntax.

<!-- 1. Please speak English, this is the language all of us can speak and write. 2. Please take a moment to check that your issue doesn't already exist. 3. Please give all relevant information below for bug reports, because incomplete details will be handled as an invalid report. --> # Aer Information - Aer Version (or commit ref): 48b2e9d1c6 - Operating System: Linux - System Architecture (eg. arm, x86_64, ...): x86_64 # Your problem description Include functions [include(), include_once(), require(), require_once()] does not check for opening and closing bracket. They can be called both: include('file.php'); and include 'file.php'; As there is no limit of using including function and such instruction can be put anywhere, IMHO it should have syntax like standard function. # Expected results include 'file.php'; should fail. # Current results no error about wrong syntax.
belliash added the
bug
label 2018-07-29 08:00:33 +02:00
Author
Owner

Should we treat this as a bug? Should we change this?
What is your opinion?

Should we treat this as a bug? Should we change this? What is your opinion?
belliash added
idea
and removed
bug
labels 2018-07-29 08:20:34 +02:00
Owner

In my opinion, syntax should be consistent. Let's do it.

In my opinion, syntax should be consistent. Let's do it.
belliash self-assigned this 2018-08-11 22:14:58 +02:00
Author
Owner

Will be done in compiler_rework branch.

Will be done in compiler_rework branch.
Author
Owner
b5fe36c14b
Author
Owner
9100bf0696
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: aerscript/Aer#28
No description provided.