Skip to content

Update too old functions and fix a lot of bugs #75

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.

Sign up for GitHub

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

Jump to bottom
Open
supercrafter333 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update too old functions and fix a lot of bugs #75

supercrafter333 wants to merge 2 commits into from

Conversation

@supercrafter333
Copy link

@supercrafter333 supercrafter333 commented Mar 1, 2021

  • {} is too old now use [] for arrays
  • implode($array, ...) ist TOO old, now use implode(..., $array)

supercrafter333 and others added 2 commits February 20, 2021 02:19
@supercrafter333
Fix Issue FaigerSYS#73
004802d 
Change $file{0} to $file[0], {} is too old for arrays
@supercrafter333
Update too old implode function and fix FaigerSYS#73 better
3036132
@supercrafter333 supercrafter333 mentioned this pull request Mar 1, 2021
Closed
@supercrafter333
Copy link
Author

supercrafter333 commented Mar 1, 2021

IT WORKS!! --- Tested

TheNewHEROBRINEX
TheNewHEROBRINEX suggested changes Jul 3, 2021
View reviewed changes
Copy link

@TheNewHEROBRINEX TheNewHEROBRINEX left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changing all ' to " for string delimiting is an unrelated change.

@TheNewHEROBRINEX
Copy link

TheNewHEROBRINEX commented Jul 7, 2021

Duplicate of #78 (which has already been merged)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@TheNewHEROBRINEX TheNewHEROBRINEX TheNewHEROBRINEX requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@supercrafter333 @TheNewHEROBRINEX