Bug 69536 - UI: Location of AVERAGEIF and AVERAGEIFS functions
Summary: UI: Location of AVERAGEIF and AVERAGEIFS functions
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
4.0.0.3 release
Hardware: All All
: medium normal
Assignee: Joel Madero
URL:
Whiteboard: BSA target:4.2.0
Keywords:
Depends on:
Blocks:
 
Reported: 2013-09-18 17:14 UTC by John Smith
Modified: 2015-12-15 12:08 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Smith 2013-09-18 17:14:18 UTC
Problem description: Both functions have been placed in the Mathematical category of the Function Wizard.

Steps to reproduce:Open an instance of Calc.
1. Open the function Wizard
2. Select the Mathematical category
3. Locate the two functions
Current behavior:Both functions have been located in the Mathematical category listing.

Expected behavior:The two functions should be located in the Statistical category with the other averaging functions AVEDEV, AVERAGE and AVERAGEA. Just as they are in MS Excel.

              
Operating System: Windows XP
Version: 4.1.1.2 release
Comment 1 tommy27 2013-09-18 21:26:01 UTC
I agree with reporter. Average functions should be grouped in the same category.

Those function were not available in 3.6.7 and earlier release while uppear under "mathematical" in 4.1.1 and older releases at least up to 4.0.4 (did not have 4.0.0 --> 4.0.3 to test but I suppose they were added in 4.0.0.

interestingly they are kisted under "mathematical" in AOO 4.0.0 as well.
Comment 2 Eike Rathke 2013-09-19 09:56:56 UTC
This was probably because AVERAGEIF and AVERAGEIFS are more like SUMIF and SUMIFS that are also in the Mathematical group.

However, as both Excel and ODF OpenFormula group them under Statistical, why not move them there..
Comment 3 Eike Rathke 2013-09-19 10:15:53 UTC
Making this an EasyHack, source code pointers:
sc/source/ui/src/scfuncs.src
sc/qa/unit/ucalc.cxx
Comment 4 Joel Madero 2013-09-19 16:30:46 UTC
I'll take it
Comment 5 Joel Madero 2013-09-23 23:00:34 UTC
Patch submitted - ready for review
Comment 6 Commit Notification 2013-09-24 00:07:43 UTC
Joel Madero committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=4547d8c42e5b0bb456ac02fbea67059a8a1cadd7

fdo#69536 - Moved Averageif(s)



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 7 Samuel Mehrbrodt (allotropia) 2013-09-29 20:01:33 UTC
Can this be closed then?
Comment 8 Joel Madero 2013-09-29 20:07:06 UTC
Fixed
Comment 9 Samuel Mehrbrodt (allotropia) 2013-09-29 20:15:43 UTC
Great :)
Comment 10 Robinson Tryon (qubit) 2015-12-15 12:08:12 UTC
Migrating Whiteboard tags to Keywords: (EasyHack,DifficultyBeginner,SkillCpp,TopicUI)
[NinjaEdit]