Make WordPress Core

Opened 12 years ago

Closed 9 years ago

#20706 closed enhancement (wontfix)

Adding a link without a name or web address yields no (error) feedback

Reported by: avreidy's profile AVReidy Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch dev-feedback
Focuses: Cc:

Description

If a user successfully creates a link (on link-add.php), the user receives the message "Link added." However, there are no measures taken to notify the user if the link was not successfully added.

Links are not added when the link name and web address fields are left blank, so I have included a little patch to let the user know when this occurs.

This could be done with some JavaScript form validation that makes an alert box pop up, but half the (PHP/HTML) code was already present for this, and I think it looks better this way, personally. This is my first attempt at submitting a patch to any open source project, so I appreciate your feedback. Thanks!

Attachments (3)

edit-link-form.php.patch (864 bytes) - added by AVReidy 12 years ago.
link.php.patch (399 bytes) - added by AVReidy 12 years ago.
20706.patch (1.1 KB) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (7)

#1 follow-up: @kawauso
12 years ago

edit-link-form.php.patch needs to retain i18n support with _e() or __()

Also both patches should be generated with paths given relative to root.

#2 in reply to: ↑ 1 @AVReidy
12 years ago

Thanks for the feedback, kawauso. I replaced the echo functions with _e(), and I regenerated the patches from the root directory (uploaded them again, as well).

Last edited 12 years ago by AVReidy (previous) (diff)

@AVReidy
12 years ago

#3 @AVReidy
12 years ago

Thanks Sergey. Still getting used to the coding standards.

#4 @chriscct7
9 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Severity changed from minor to normal
  • Status changed from new to closed

Links are no longer available on new installs since 3.5

Note: See TracTickets for help on using tickets.