Popular Languages listing: Don't show variations #13

Closed
opened 2018-11-17 07:34:38 +01:00 by AndyHee · 3 comments
AndyHee commented 2018-11-17 07:34:38 +01:00 (Migrated from github.com)

In the nice, new listing "Popular Languages", we have English, American English, and British English.

Clicking English: i.e. https://dir.hubup.pro/search?field=language&q=en list all variations of English, including American and British. This is meaningful and desirable, but makes the listing of language variations superfluous.

Not displaying variations would be an enhancement, as it would free up space for genuine languages with lower numbers to be displayed, especially since we already have "Popular Country" listings.

I guess the undesirable display of language variations, could be fixed by limiting the field in relevant table to the first two entries.

In the nice, new listing "Popular Languages", we have English, American English, and British English. Clicking English: i.e. https://dir.hubup.pro/search?field=language&q=en list all variations of English, including American and British. This is meaningful and desirable, but makes the listing of language variations superfluous. Not displaying variations would be an enhancement, as it would free up space for genuine languages with lower numbers to be displayed, especially since we already have "Popular Country" listings. I guess the undesirable display of language variations, could be fixed by limiting the field in relevant table to the first two entries.
MrPetovan commented 2018-11-17 12:48:22 +01:00 (Migrated from github.com)

I agree, I've been meaning to dpo the same, thanks for reporting! The fix is simple, limiting the aggregation list to the first two characters of the profile locale.

I agree, I've been meaning to dpo the same, thanks for reporting! The fix is simple, limiting the aggregation list to the first two characters of the profile locale.
AndyHee commented 2018-11-18 05:24:30 +01:00 (Migrated from github.com)

I took note how you did this: 01d476f542?diff=unified Looks fascinating; I think, I'm beginning to grasp --at a very basic level-- how php interacts with the DB.

I took note how you did this: https://github.com/friendica/friendica-directory/commit/01d476f5427c650eb51750e348b6df2ee2a245c9?diff=unified Looks fascinating; I think, I'm beginning to grasp --at a very basic level-- how php interacts with the DB.
MrPetovan commented 2018-11-18 05:45:27 +01:00 (Migrated from github.com)

This is indeed pure SQL. PHP only grabs the results from the SQL query and pass it to the template where it is displayed.

This is indeed pure SQL. PHP only grabs the results from the SQL query and pass it to the template where it is displayed.
Sign in to join this conversation.
No Milestone
No Assignees
1 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: friendica/friendica-directory#13
No description provided.