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

[com_fields] Wrong frontend fields ordering #16183

Merged
merged 1 commit into from
May 22, 2017
Merged

Conversation

hieblmedia
Copy link
Contributor

@hieblmedia hieblmedia commented May 22, 2017

Summary of Changes

In the frontend the field ordering is reversed to the backend, because DESC is default instead of ASC.

Testing Instructions

Create two or more fields and reorder it.

Expected result

Frontend and Backend ordering are equal.

Actual result

Frontend ordering is reversed to the backend ordering

In the frontend the field ordering is reversed, because DESC is default instead of ASC.
@ghost
Copy link

ghost commented May 22, 2017

Didn't get Issue:

Fields for Articles, alle Fields in same Order:

1

Fields in Article:

2

Frontend:

3

System information

3.7.2-dev
macOS Sierra, 10.12.4
Firefox 53 (64-bit)

MAMP 4.1.1

  • PHP 7.0.15
  • MySQLi 5.6.35

@brianteeman
Copy link
Contributor

I can confirm that the field ordering is messed up. It is fine if you just want to display the fields in the order that they are entered but as soon as you change the ordering then it breaks,
fields

@Quy
Copy link
Contributor

Quy commented May 22, 2017

I have tested this item ✅ successfully on 6b1c508


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16183.

1 similar comment
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 6b1c508


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16183.

@ghost
Copy link

ghost commented May 22, 2017

RTC after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 22, 2017
@brianteeman brianteeman added this to the Joomla 3.7.3 milestone May 22, 2017
@rdeutz rdeutz merged commit 2fb1620 into joomla:staging May 22, 2017
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels May 22, 2017
@Quy Quy mentioned this pull request May 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants