Skip to content
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.

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

[BUG] PHP Error with Add More Records #423

Closed
sheepsong opened this issue Dec 13, 2018 · 1 comment
Closed

[BUG] PHP Error with Add More Records #423

sheepsong opened this issue Dec 13, 2018 · 1 comment

Comments

@sheepsong
Copy link

@sheepsong sheepsong commented Dec 13, 2018

fM Version : 3.2.0
fmDNS Version : 3.2.0

In raising this issue, I confirm the following (please check boxes, eg [X]):

  • I have read and understood the contributors guide.
  • I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
  • I have checked that the issue I'm posting isn't already reported.
  • I have checked that the issue I'm posting isn't already solved and no duplicates exist in closed issues and opened issues
  • [] I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.

**(BUG | ISSUE) Expected Behavior:
Show more lines for input when clicking "Add more records" button in Zones as records inputs above.
**

**(BUG | ISSUE) Actual Behavior:
Showing lines without tr tags and notice undefined variable record_ptr_id in fm-modules/fmDNS/classes/class_records.php on line 464
**

**(BUG | ISSUE) Steps to reproduce:
Just click on button Add more records
**

Showing lines without tr tags and notice undefined variable record_ptr_id in fm-modules/fmDNS/classes/class_records.php on line 464.

463    /** Linked PTR */
464    				if ($record_ptr_id) {
465    					$field_values['data']['Value'] .= ' <a href="#" class="tooltip-right" data-tooltip="' . __('Linked PTR exists') . '"><i class="mini-icon fa fa-exchange"></i></a>';
466    				}

Maybe it is need to be sure $record_ptr_id is defined.

@WillyXJ WillyXJ changed the title [BUG] [BUG] PHP Error with Add More Records Dec 13, 2018
WillyXJ pushed a commit that referenced this issue Dec 13, 2018
@WillyXJ WillyXJ added the fmDNS label Dec 13, 2018
@WillyXJ
Copy link
Owner

@WillyXJ WillyXJ commented Dec 15, 2018

This is now fixed in fmDNS 3.2.1 and later.

@WillyXJ WillyXJ closed this Dec 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants